Skip to content
This repository was archived by the owner on Dec 23, 2021. It is now read-only.

HHVM-193: executeBulkWrite() should throw InvalidArgumentException for empty BulkWrite #77

Merged
merged 1 commit into from
Mar 22, 2016

Conversation

derickr
Copy link
Contributor

@derickr derickr commented Mar 17, 2016

No description provided.

$m = new MongoDB\Driver\Manager("mongodb://localhost:27017");
cleanup( $m );

$bw = new MongoDB\Driver\BulkWrite( [ 'ordered' => false ] );
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is [ 'ordered' => false ] relevant? I'd think to just leave the ctor arg empty for conciseness.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, not relevant. I just copied it from somewhere else. I'll fix it.

@jmikola
Copy link
Member

jmikola commented Mar 17, 2016

LGTM pending question above.

@derickr derickr force-pushed the HHVM-193-empty-bulkwrite-invalidargumentexception branch from 8e01db2 to 567c0af Compare March 17, 2016 18:28
@derickr derickr merged commit 567c0af into master Mar 22, 2016
derickr added a commit that referenced this pull request Mar 22, 2016
derickr added a commit that referenced this pull request Mar 22, 2016
@derickr derickr deleted the HHVM-193-empty-bulkwrite-invalidargumentexception branch March 23, 2016 00:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants