Skip to content
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

[Lock] #37139 Updated Lock MongoDbStore note on readPreference #13775

Merged

Conversation

kralos
Copy link

@kralos kralos commented Jun 8, 2020

Fixed documentation around readPreference for MongoDbStore in Lock component. See symfony/symfony#37140

@javiereguiluz javiereguiluz added Lock Waiting Code Merge Docs for features pending to be merged labels Jun 8, 2020
fabpot added a commit to symfony/symfony that referenced this pull request Jun 10, 2020
…s (kralos)

This PR was squashed before being merged into the 5.1 branch.

Discussion
----------

[Lock] Fixed reading locks from replica set secondary nodes

| Q             | A
| ------------- | ---
| Branch?       | 5.1
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Tickets       | Fix #37139
| License       | MIT
| Doc PR        | symfony/symfony-docs#13775

Force lock existence query to use `readPreference=primary` in case the given mongo connection is using any of the following `readPreference`s:

* primaryPreferred
* secondary
* secondaryPreferred
* nearest

Any of the above would fail if a secondary node is queried during a lock release.

Commits
-------

ebf7eaf [Lock] Fixed reading locks from replica set secondary nodes
symfony-splitter pushed a commit to symfony/lock that referenced this pull request Jun 10, 2020
…s (kralos)

This PR was squashed before being merged into the 5.1 branch.

Discussion
----------

[Lock] Fixed reading locks from replica set secondary nodes

| Q             | A
| ------------- | ---
| Branch?       | 5.1
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Tickets       | Fix #37139
| License       | MIT
| Doc PR        | symfony/symfony-docs#13775

Force lock existence query to use `readPreference=primary` in case the given mongo connection is using any of the following `readPreference`s:

* primaryPreferred
* secondary
* secondaryPreferred
* nearest

Any of the above would fail if a secondary node is queried during a lock release.

Commits
-------

ebf7eaf503 [Lock] Fixed reading locks from replica set secondary nodes
@xabbuh xabbuh removed the Waiting Code Merge Docs for features pending to be merged label Oct 16, 2020
@carsonbot carsonbot changed the title #37139 Updated Lock MongoDbStore note on readPreference [Lock] #37139 Updated Lock MongoDbStore note on readPreference Jan 27, 2021
@javiereguiluz javiereguiluz added this to the 5.1 milestone Jan 27, 2021
@javiereguiluz
Copy link
Member

Thank you Joe! Sorry it took us so long to merge this.

@javiereguiluz javiereguiluz merged commit e905ac8 into symfony:5.1 Jan 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants