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

Fix move_jumbo_chunk.js to not require disabling the resumable range deleter

    • Type: Icon: Task Task
    • Resolution: Unresolved
    • Priority: Icon: Major - P3 Major - P3
    • None
    • Affects Version/s: None
    • Component/s: Sharding
    • Cluster Scalability
    • Sharding 2020-12-14, Sharding 2020-12-28, Sharding 2021-01-11, Sharding 2021-01-25, Sharding 2021-02-22, Sharding 2021-03-08, Sharding 2021-03-22, Sharding 2021-04-05, Sharding 2021-04-19, Sharding 2021-05-03

      This test requires the resumable range deleter to be disabled in order to run. This is because it sets a failpoint to make the recipient shard hang, which means that advanceTransactionOnRecipient will hang on the recipient trying to check out the session, which will prevent the migration from completing.

      Unfortunately this is hard to fix in this test because we're using the balancer to do the migration. In addition, we can't avoid setting the migrateThreadHangAtStep2 failpoint because then this call to the recipient could yield Status::OK(), bypassing the error we're trying to hit in the test.

      I'm not sure what a good solution is, but fixing this would be required to remove the disableResumableRangeDeleter flag in 4.6.

      As a side note for whoever looks at this later, it is not necessary to test three rounds of the balancer and in fact this behavior is incorrect - the first time it will fail due to ExceededMemoryLimit as expected, but the second two migration attempts fail due to ConflictingOperationInProgress since the recipient remains hung at the failpoint even after the first migration fails.

            Assignee:
            backlog-server-cluster-scalability [DO NOT USE] Backlog - Cluster Scalability
            Reporter:
            matthew.saltz@mongodb.com Matthew Saltz (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated: