[SERVER-53811] abortTransaction on mongos fails with BSON field 'abortTransaction.recoveryToken' is an unknown field Created: 14/Jan/21 Updated: 29/Oct/23 Resolved: 20/Jan/21 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Replication, Sharding |
| Affects Version/s: | None |
| Fix Version/s: | 4.9.0-alpha2 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Shane Harvey | Assignee: | Moustafa Maher |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||
| Backwards Compatibility: | Minor Change | ||||||||||||
| Operating System: | ALL | ||||||||||||
| Sprint: | Repl 2021-01-25 | ||||||||||||
| Participants: | |||||||||||||
| Description |
|
Running abortTransaction on 4.9 mongos with a recoveryToken results in the following error:
The full command was:
The server version:
This is unexpected because drivers send recoveryToken on both commitTransaction and abortTransaction as decided in |
| Comments |
| Comment by Moustafa Maher [ 20/Jan/21 ] | |||||||||||||||||
|
recoveryToken has been addd as accepted optional field for abortTransaction | |||||||||||||||||
| Comment by Githook User [ 20/Jan/21 ] | |||||||||||||||||
|
Author: {'name': 'Moustafa Maher', 'email': 'm.maher@10gen.com', 'username': 'moustafamaher'}Message: | |||||||||||||||||
| Comment by Moustafa Maher [ 19/Jan/21 ] | |||||||||||||||||
|
shane.harvey I have checked, and no Server doesn't run the driver tests, but it will be ran after the code is committed to master on the driver side. | |||||||||||||||||
| Comment by Shane Harvey [ 19/Jan/21 ] | |||||||||||||||||
|
m.maher, does the server implement the driver transaction spec tests? Specifically these 4 tests started failing because of this change: https://github.com/mongodb/specifications/blob/d05cbdf/source/transactions/tests/retryable-abort.yml#L1017-L1311 You can see the test failures in | |||||||||||||||||
| Comment by Shane Harvey [ 19/Jan/21 ] | |||||||||||||||||
|
m.maher, please refer to https://github.com/mongodb/specifications/blob/master/source/transactions/transactions.rst#server-commands :
| |||||||||||||||||
| Comment by Kaloian Manassiev [ 15/Jan/21 ] | |||||||||||||||||
|
Assigning it to the Replication Team's backlog. CC m.maher. | |||||||||||||||||
| Comment by Kaloian Manassiev [ 15/Jan/21 ] | |||||||||||||||||
|
I am also surprised it is not failing tests like this one which explicitly send the recovery token. | |||||||||||||||||
| Comment by Shane Harvey [ 14/Jan/21 ] | |||||||||||||||||
|
I suspect this was caused by |