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

Check DatabaseShardingState for index build commit under an exception handler

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major - P3
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.4.0-rc5, 4.7.0
    • Component/s: None
    • Labels:
      None
    • Backwards Compatibility:
      Fully Compatible
    • Operating System:
      ALL
    • Backport Requested:
      v4.4
    • Sprint:
      Execution Team 2020-05-18
    • Linked BF Score:
      30

      Description

      This invariant makes an assumption that index builds will not throw database exceptions except for when they commit, and we hold an X collection lock.

      I thought I covered every exception-throwing region of index builds, but I think I missed a spot. We should move this DatabaseShardingState version check inside the subsequent try-catch block. We're already holding an X lock there so it will abort the index build as usual if the assertion fails.

        Attachments

          Activity

            People

            Assignee:
            louis.williams Louis Williams
            Reporter:
            louis.williams Louis Williams
            Participants:
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: