Skip to content

Fix an old exclude failed bug#13645

Open
jzhou77 wants to merge 1 commit into
apple:mainfrom
jzhou77:fix_exclude_failed
Open

Fix an old exclude failed bug#13645
jzhou77 wants to merge 1 commit into
apple:mainfrom
jzhou77:fix_exclude_failed

Conversation

@jzhou77

@jzhou77 jzhou77 commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

This ensures serverKey map assignment is correct. Without the fix, actors for emptyRangeActors() race with actors that unassign servers, the results depend on the read order. With the fix, ranges are unassigned first, then assigned as empty.

Exclude failed command may corrupt serverKey map. This was found in simulation's DataLossRecovery test and I cherry-pick it for upstream.

This ensures serverKey map assignment is correct.
@jzhou77 jzhou77 force-pushed the fix_exclude_failed branch from 08e09b6 to 7fa5321 Compare July 3, 2026 18:29

@saintstack saintstack left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM and also to my 'replacement':

## Correctness assessment

The fix is correct, minimal, and introduces no new defect:

- `actors` / `emptyRangeActors` are freshly scoped per loop iteration.
- `waitForAll` on an empty vector is a no-op (safe when `dest` is empty).
- Phase 2's stale snapshot reads (native `Transaction`, no RYW) don't matter,
  because its writes fully overwrite the range for that `id`.

You are marked as a 'contributor' @jzhou77 ?

@foundationdb-ci

Copy link
Copy Markdown
Contributor

Result of foundationdb-pr-clang-ide on Linux RHEL 9

  • Commit ID: 7fa5321
  • Duration 0:21:26
  • Result: ✅ SUCCEEDED
  • Error: N/A
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

@foundationdb-ci

Copy link
Copy Markdown
Contributor

Result of foundationdb-pr-macos-m1 on macOS 14.x

  • Commit ID: 7fa5321
  • Duration 0:32:37
  • Result: ✅ SUCCEEDED
  • Error: N/A
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

@foundationdb-ci

Copy link
Copy Markdown
Contributor

Result of foundationdb-pr-clang-arm on Linux RHEL 9

  • Commit ID: 7fa5321
  • Duration 0:46:47
  • Result: ✅ SUCCEEDED
  • Error: N/A
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

@foundationdb-ci

Copy link
Copy Markdown
Contributor

Result of foundationdb-pr-macos on macOS 14.x

  • Commit ID: 7fa5321
  • Duration 0:47:01
  • Result: ✅ SUCCEEDED
  • Error: N/A
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

@foundationdb-ci

Copy link
Copy Markdown
Contributor

Result of foundationdb-pr-clang on Linux RHEL 9

  • Commit ID: 7fa5321
  • Duration 0:51:10
  • Result: ❌ FAILED
  • Error: Error while executing command: if python3 -m joshua.joshua list --stopped | grep ${ENSEMBLE_ID} | grep -q 'pass=10[0-9][0-9][0-9]'; then echo PASS; else echo FAIL && exit 1; fi. Reason: exit status 1
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

@foundationdb-ci

Copy link
Copy Markdown
Contributor

Result of foundationdb-pr on Linux RHEL 9

  • Commit ID: 7fa5321
  • Duration 0:55:04
  • Result: ✅ SUCCEEDED
  • Error: N/A
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

@foundationdb-ci

Copy link
Copy Markdown
Contributor

Result of foundationdb-pr-cluster-tests on Linux RHEL 9

  • Commit ID: 7fa5321
  • Duration 4:00:18
  • Result: ❌ FAILED
  • Error: Error while executing command: if $fail_test; then exit 1; fi. Reason: exit status 1
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)
  • Cluster Test Logs zip file of the test logs (available for 30 days)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants