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

ChangeStreams lookup_post_image.js test makes assumptions that don't hold up in secondary read passthrough

    XMLWordPrintable

    Details

    • Backwards Compatibility:
      Fully Compatible
    • Operating System:
      ALL
    • Backport Requested:
      v3.6
    • Sprint:
      Repl 2017-12-18
    • Linked BF Score:
      0

      Description

      The lookup_post_image.js test does an update to a document, then immediately removes it, the advances a changeStream to consume the notification corresponding to the update and expects the looked up fullDocument from the updateLookup to be null since the doc was deleted. This assertion is valid if the read and write go to the same node, but in the change_streams_secondary_reads suite this assumption is invalid, leading to the test to be flaky. We should either modify the test to not make this incorrect assumption, or blacklist it from the change_streams_secondary_reads suite.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              matthew.russotto Matthew Russotto
              Reporter:
              spencer Spencer Brody
              Participants:
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: