[SERVER-56550] Require Versioned API options for getMore and transaction-continuing commands Created: 01/May/21 Updated: 29/Oct/23 Resolved: 16/May/21 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 5.0.0-rc0 |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | A. Jesse Jiryu Davis | Assignee: | A. Jesse Jiryu Davis |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||||||
| Backwards Compatibility: | Fully Compatible | ||||||||||||||||||||||||
| Sprint: | Elastic 2021-05-10, Elastic 2021-05-17 | ||||||||||||||||||||||||
| Participants: | |||||||||||||||||||||||||
| Description |
|
Once the dust settles on |
| Comments |
| Comment by A. Jesse Jiryu Davis [ 16/May/21 ] |
|
I was confused, we upgrade mongos after shards, so the upgrade bug I was worried about can't actually be a problem: https://docs.mongodb.com/manual/release-notes/4.4-upgrade-sharded-cluster/ |
| Comment by Githook User [ 13/May/21 ] |
|
Author: {'name': 'A. Jesse Jiryu Davis', 'email': 'jesse@mongodb.com', 'username': 'ajdavis'}Message: Transaction-continuing commands must use the same API parameters as the |
| Comment by Githook User [ 13/May/21 ] |
|
Author: {'name': 'A. Jesse Jiryu Davis', 'email': 'jesse@mongodb.com', 'username': 'ajdavis'}Message: |
| Comment by A. Jesse Jiryu Davis [ 07/May/21 ] |
|
Before we close this, let's talk about upgrade/downgrade issues. I think there's an issue with mongos 5.0 and mongod 4.4. A client can call commitTransaction on mongos with apiVersion: "1", and mongos calls prepareTransaction on shards with apiVersion: "1", they respond with an unrecognized-parameter error, and mongos retries forever(?). |
| Comment by A. Jesse Jiryu Davis [ 01/May/21 ] |
|
Restore the CannotSpecifyAPIParametersAfterFirstStatement test I deleted from transaction_router_test.cpp in |