Uploaded image for project: 'Core Server'
  1. Core Server
  2. SERVER-33640

Blacklist find_and_modify_concurrent_update.js from causal consistency passthrough

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major - P3
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.6.4, 3.7.3
    • Component/s: Testing Infrastructure
    • Labels:
      None
    • Backwards Compatibility:
      Fully Compatible
    • Backport Requested:
      v3.6
    • Sprint:
      Query 2018-03-12
    • Linked BF Score:
      0

      Description

      This test uses a parallel shell, and (in this passthrough) sometimes expects a future read to be causally consistent with the other shell's update. This is not a valid assumption.

      While we could re-write the test in a way that would allow the main thread to wait and observe the parallel shell's write, it doesn't seem worth it to modify the test and gain that coverage. If such coverage of parallel shells and causal consistency is desired, it should probably be written as it's own standalone test.

      As part of this, we should do a quick scan to see if there are any other tests that might expect causal consistency with a parallel shell.

        Attachments

          Activity

            People

            Assignee:
            charlie.swanson Charlie Swanson
            Reporter:
            charlie.swanson Charlie Swanson
            Participants:
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: