SERVER-17953 Implement Driver CRUD API Spec in shell#1004
Closed
christkv wants to merge 39 commits intomongodb:masterfrom
Closed
SERVER-17953 Implement Driver CRUD API Spec in shell#1004christkv wants to merge 39 commits intomongodb:masterfrom
christkv wants to merge 39 commits intomongodb:masterfrom
Conversation
This output can be used to run previous schedules when debugging test failures.
No functional changes, just moving code around.
Currently the write commands explain code calls ensureShardVersionOKOrThrow, which performs legacy style shard version checking based on the contents of the connection. This however does not have any effect, because write commands are not sent over versioned connections, so effectively write command explain is non-sharding aware currently. This change removes this check for the explain commands.
…r in unit tests. The fetcher test is excluded at present, because it appears to depend on some implementation behavior of ReplicationExecutor that remains to be diagnosed.
With this change all legacy-style (non-NetworkInterface) communication from mongos to shard mongod is going through ShardConnection and will always end the shard initialization data at connection establishment.
…log using Fetcher instead of OplogReader
…command line to resmoke.py
link the option for callers of ::exec
o run JS_MaybeGC in the interrupt handler o run manual gc() in DBQuery.itcount()
… check for network errors
…ing a collection to force the version to be set
Use built-in ES5 function for less error-prone array detection and limit the number of array elements to 3.
mmou
pushed a commit
to mmou/mongo
that referenced
this pull request
Aug 6, 2015
Closes mongodb#1004. Signed-off-by: Max Hirschhorn <[email protected]> - Moved CRUD API implementation into separate file to workaround 65K string length compiler limit on Windows. - Fixed test to ignore modifiedCount property (i.e. nModified) when not using write commands.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.