Skip to content

SERVER-17953 Implement Driver CRUD API Spec in shell#1004

Closed
christkv wants to merge 39 commits intomongodb:masterfrom
christkv:SERVER-17953
Closed

SERVER-17953 Implement Driver CRUD API Spec in shell#1004
christkv wants to merge 39 commits intomongodb:masterfrom
christkv:SERVER-17953

Conversation

@christkv
Copy link
Copy Markdown
Contributor

No description provided.

ramonfm and others added 30 commits July 20, 2015 16:32
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.
link the option for callers of ::exec
o run JS_MaybeGC in the interrupt handler
o run manual gc() in DBQuery.itcount()
@visemet visemet closed this in 8c8da71 Aug 4, 2015
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.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.