[SERVER-71219] Migration can miss writes from prepared transactions Created: 09/Nov/22 Updated: 29/Oct/23 Resolved: 27/Jan/23 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Sharding |
| Affects Version/s: | 4.2.0, 4.4.0, 5.0.0, 6.0.0, 6.1.0-rc3 |
| Fix Version/s: | 4.4.19, 5.0.15, 6.3.0-rc0, 6.0.5 |
| Type: | Bug | Priority: | Critical - P2 |
| Reporter: | Randolph Tan | Assignee: | Randolph Tan |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||||||
| Backwards Compatibility: | Fully Compatible | ||||||||||||||||||||||||
| Operating System: | ALL | ||||||||||||||||||||||||
| Backport Requested: |
v6.2, v6.0, v5.0, v4.4
|
||||||||||||||||||||||||
| Steps To Reproduce: | Uncomment this line in test |
||||||||||||||||||||||||
| Sprint: | Sharding NYC 2022-11-28, Sharding 2022-12-12, Sharding NYC 2022-12-26, Sharding NYC 2023-01-09, Sharding NYC 2023-01-23, Sharding NYC 2023-02-06 | ||||||||||||||||||||||||
| Participants: | |||||||||||||||||||||||||
| Linked BF Score: | 113 | ||||||||||||||||||||||||
| Description |
|
Migration relies on LogTransactionOperationsForShardingHandler to capture writes that gets committed. However, the issue is that the commit handler only gets registered to the recovery unit during prepare. This means that if another node becomes the primary, no callbacks from LogTransactionOperationsForShardingHandler will get called when the transaction gets committed. |
| Comments |
| Comment by Githook User [ 08/Feb/23 ] |
|
Author: {'name': 'Randolph Tan', 'email': 'randolph@10gen.com', 'username': 'renctan'}Message: This commit does 2 things: (cherry picked from commit a20d97c8e30b805410d86ccdbeac2d3f05c407ba) |
| Comment by Githook User [ 08/Feb/23 ] |
|
Author: {'name': 'Randolph Tan', 'email': 'randolph@10gen.com', 'username': 'renctan'}Message: This commit does 2 things: (cherry picked from commit bd227ce5529b0423ca791aa926db48ec8b10c07d) |
| Comment by Githook User [ 03/Feb/23 ] |
|
Author: {'name': 'Randolph Tan', 'email': 'randolph@10gen.com', 'username': 'renctan'}Message: This commit does 2 things: (cherry picked from commit 2f708612dc39780410bf40d31a404cb121f653a9) |
| Comment by Githook User [ 27/Jan/23 ] |
|
Author: {'name': 'Randolph Tan', 'email': 'randolph@10gen.com', 'username': 'renctan'}Message: This commit does 2 things: |
| Comment by Githook User [ 27/Jan/23 ] |
|
Author: {'name': 'Randolph Tan', 'email': 'randolph@10gen.com', 'username': 'renctan'}Message: |