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

TransactionCoordinatorMetricsTest should install mock tick source before starting WaitForMajorityService

    XMLWordPrintable

    Details

    • Backwards Compatibility:
      Fully Compatible
    • Operating System:
      ALL
    • Sprint:
      Sharding 2020-06-15, Sharding 2020-06-29
    • Linked BF Score:
      0

      Description

      TransactionCoordinatorMetricsTest::setUp() sets a MockTickSource on the service context after calling TransactionCoordinatorTestBase::setUp(). This sets up the TransactionCoordinatorTestFixture, which begins the WaitForMajorityService, which spawns a thread that creates operation contexts, which accesses the service context's tick source. The tick source isn't protected by a mutex, so this is a data race. Setting the mock tick source before setting up the WaitForMajorityService should avoid this.

        Attachments

          Activity

            People

            Assignee:
            luis.osta Luis Osta Lugo (Inactive)
            Reporter:
            jack.mulrow Jack Mulrow
            Participants:
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: