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: