Skip to content

PHPC-1369: Upgrade libmongoc to 1.15-dev #996

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Jul 23, 2019
Merged

Conversation

jmikola
Copy link
Member

@jmikola jmikola commented Jul 19, 2019

@jmikola jmikola force-pushed the phpc-1369 branch 6 times, most recently from e202781 to 49fe725 Compare July 22, 2019 20:19
@jmikola jmikola requested a review from alcaeus July 22, 2019 20:38
jmikola added 4 commits July 23, 2019 09:38
Use int64_t API for wTimeoutMS. The initial commit for CDRIVER-3087 deprecated the int32_t API for wTimeoutMS. Migrate to the int64_t API to avoid deprecation warnings. Note: PHPC (and libmongoc's URI parsing) still require 32-bit values for wTimeoutMS, so this does not affect users. Having said that, PHPC-1411 may introduce support for 64-bit values and necessitate changes to the code where we read wTimeoutMS values.

Use mongoc_uri_get_tls to read tls/ssl URI option. As part of CDRIVER-2869, libmongoc deprecated mongoc_uri_get_ssl and introduced a "tls" alias. Migrate to the new API to avoid deprecation warnings.

Expect lowercase URI option in error. As part of CDRIVER-2869, libmongoc does case-folding on option keys to facilitate deduplication.
Additionally, we can use the generated collection name instead of "example".
Without replica set shards, retryable writes cannot be used. It must be explicitly disabled because the driver cannot detect this by its direct communication with mongos nodes.
@jmikola jmikola merged commit b47adb7 into mongodb:master Jul 23, 2019
jmikola added a commit that referenced this pull request Jul 23, 2019
@jmikola jmikola deleted the phpc-1369 branch July 23, 2019 13:40
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.

2 participants