[SERVER-8070] Flush buffer before changing sync targets to prevent unnecessary rollbacks Created: 03/Jan/13 Updated: 22/Sep/17 Resolved: 18/Jan/13 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Replication |
| Affects Version/s: | 2.2.2 |
| Fix Version/s: | 2.4.0-rc0, 2.5.0 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Kristina Chodorow (Inactive) | Assignee: | Eric Milkie |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||
| Operating System: | ALL | ||||
| Participants: | |||||
| Case: | (copied to CRM) | ||||
| Linked BF Score: | 0 | ||||
| Description |
|
The problem is with bgsync.cpp here:
If we're syncing from the primary, then this logic is valid. If we are not, then this member must have applied some ops from the buffer and got ahead of the sync source, which is okay and we don't want to rollback. Probably the right fix is to make sure the buffer is empty before choosing a new sync target. This is a fairly annoying bug to run into. If the member is in the middle of applying a batch of ops, the rollback will fail (because you're not at minvalid) and put the member into FATAL state. (I think restarting should fix it, though.) |
| Comments |
| Comment by auto [ 23/May/13 ] |
|
Author: {u'username': u'milkie', u'name': u'Eric Milkie', u'email': u'milkie@10gen.com'}Message: |
| Comment by auto [ 22/May/13 ] |
|
Author: {u'username': u'milkie', u'name': u'Eric Milkie', u'email': u'milkie@10gen.com'}Message: |
| Comment by auto [ 10/May/13 ] |
|
Author: {u'date': u'2013-05-10T20:09:18Z', u'name': u'Eric Milkie', u'email': u'milkie@10gen.com'}Message: |
| Comment by auto [ 10/May/13 ] |
|
Author: {u'date': u'2013-05-10T15:57:27Z', u'name': u'Eric Milkie', u'email': u'milkie@10gen.com'}Message: |
| Comment by auto [ 28/Jan/13 ] |
|
Author: {u'date': u'2013-01-28T14:20:52Z', u'email': u'kristina@10gen.com', u'name': u'Kristina'}Message: |
| Comment by auto [ 15/Jan/13 ] |
|
Author: {u'date': u'2013-01-15T20:59:43Z', u'email': u'kristina@10gen.com', u'name': u'Kristina'}Message: |
| Comment by auto [ 15/Jan/13 ] |
|
Author: {u'date': u'2013-01-15T19:21:52Z', u'email': u'kristina@10gen.com', u'name': u'Kristina'}Message: |