shouldRetry() logic in multiInitialSyncApply_noAbort() is over-aggressive

XMLWordPrintableJSON

    • Type: Bug
    • Resolution: Done
    • Priority: Major - P3
    • 3.4.7, 3.5.10
    • Affects Version/s: None
    • Component/s: Replication
    • None
    • Fully Compatible
    • ALL
    • v3.4
    • Repl 2017-05-08, Repl 2017-05-29, Repl 2017-06-19, Repl 2017-07-31
    • None
    • 3
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      Currently multiInitialSyncApply_noAbort() calls shouldRetry() on all !OK statuses returned from applying any non-command op. If shouldRetry() returns false, the error is ignored. It should probably only do this for error cases that are expecting the shouldRetry() treatment (such as an update not finding a document) rather than for absolutely all errors.

              Assignee:
              Judah Schvimer
              Reporter:
              Mathias Stearn
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

                Created:
                Updated:
                Resolved: