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

Await all operations committed in ShardingTest::checkUUIDsConsistent hook (lookup.js)

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major - P3
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.7.6
    • Component/s: Sharding
    • Labels:
      None
    • Backwards Compatibility:
      Fully Compatible
    • Operating System:
      ALL
    • Sprint:
      Sharding 2018-04-23
    • Linked BF Score:
      56

      Description

      jstests/aggregation/sources/lookup/lookup.js manually creates a ShardingTest, and because it can run in a read concern majority passthrough, the checkUUIDs hook can be overriden to run with majority read concern, which it does not support because the metadata collections it reads from are written with local write concern. SERVER-33876 made ShardingTest default to 1 node replica sets instead of standalone nodes, which exposes this problem because now locally written data is not guaranteed to be immediately seen by majority reads.

      The checkUUIDs hook should await all operations to be committed before checking the shard metadata collections.

        Attachments

          Activity

            People

            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: