[SERVER-43978] Stable timestamp is not being recalculated after aborting oplog holes Created: 12/Oct/19 Updated: 29/Oct/23 Resolved: 12/Nov/19 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Replication |
| Affects Version/s: | None |
| Fix Version/s: | 4.3.1, 4.2.3 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Lingzhi Deng | Assignee: | William Schultz (Inactive) |
| 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: |
v4.2
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Steps To Reproduce: |
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Sprint: | Repl 2019-11-04, Repl 2019-11-18 | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Participants: | |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Linked BF Score: | 7 | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Description |
|
If we reserve an oplog slot but decide to abort later, the stable timestamp is not being recalculated even though the oplog hole has been released. This is similar to the bug described in (Note: commit seems fine because we always set my lastApplied in _logOpsInner, which will update stable timestamp). |
| Comments |
| Comment by Githook User [ 06/Jan/20 ] | ||||||||||||||||||||||||||||||||
|
Author: {'name': 'William Schultz', 'email': 'william.schultz@mongodb.com', 'username': 'will62794'}Message: | ||||||||||||||||||||||||||||||||
| Comment by Githook User [ 06/Jan/20 ] | ||||||||||||||||||||||||||||||||
|
Author: {'name': 'William Schultz', 'email': 'william.schultz@mongodb.com', 'username': 'will62794'}Message: (cherry picked from commit 754c07c70cf5cd3c6760683bc29c927010a5718a) | ||||||||||||||||||||||||||||||||
| Comment by Githook User [ 12/Nov/19 ] | ||||||||||||||||||||||||||||||||
|
Author: {'name': 'William Schultz', 'username': 'will62794', 'email': 'william.schultz@mongodb.com'}Message: | ||||||||||||||||||||||||||||||||
| Comment by William Schultz (Inactive) [ 07/Nov/19 ] | ||||||||||||||||||||||||||||||||
|
lingzhi.deng raised the question of whether the fix for this ticket may make the changes from | ||||||||||||||||||||||||||||||||
| Comment by William Schultz (Inactive) [ 22/Oct/19 ] | ||||||||||||||||||||||||||||||||
|
From a quick search against this commit, the existing callers of repl::getNextOpTimes() are:
and the callers of repl::getNextOpTime:
| ||||||||||||||||||||||||||||||||
| Comment by Judah Schvimer [ 14/Oct/19 ] | ||||||||||||||||||||||||||||||||
|
In practice, the noop writer may make this unlikely, but we'll fix it to fix the BFs. |