Flaky on Windows: attach_on_missing_create_is_idempotent_for_existing_file (attach_tests.rs:312)
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 48/100
- Issue type
- Bug
- Clarity
- Mostly clear
- Activity status
- Active
- Tech stack
- github-actions, rust
- Domain
- ci-cd, databases, operating-systems, testing
Research direction
Start with hyperdb-mcp/tests/attach_tests.rs:312 and repeatedly run the Windows CI leg on an unchanged tree to establish whether the failure recurs. Read the adjacent locking behavior in hyperdb-mcp/src/export.rs:439-442 and the analysis from #281, then determine whether file-handle release or delete-error classification explains the assertion; done means the cause is addressed and the Windows test is reliable.
Written by the indexing model from the issue text.
Description
Symptom
attach_on_missing_create_is_idempotent_for_existing_file failed on the Windows leg of a main CI run, at hyperdb-mcp/tests/attach_tests.rs:312:10:
test result: FAILED. 24 passed; 1 failed; 0 ignored; 0 measured; 0 filtered out; finished in 60.48s
- Run: 34058959366, job
test (windows-latest)(id101555912364), at commit56bc0d0. - The same run's
test (ubuntu-latest)leg also failed, but on a different test (engine_recovers_after_hyperd_killed) — that one is now fixed by #286 / #284. This issue is only about the Windowsattach_testsfailure, which is unrelated.
It appears intermittent
test (windows-latest) passed on the next main run (34062593527, commit 61fc766) with no code change touching attach.rs. So this is one observation, not a reproducible break, and it is recorded here so the evidence is not lost if it recurs.
Why it is worth investigating rather than dismissing
The neighbouring Windows behaviour in this area is genuinely different from Unix: Windows blocks concurrent opens of a locked .hyper file (see the comment at hyperdb-mcp/src/export.rs:439-442), and the attach/CREATE DATABASE IF NOT EXISTS path is exactly where that difference shows up. An idempotency test failing only on Windows is therefore consistent with a real platform-specific race around file creation or locking, not just runner noise.
Related context: the review of #281 concluded that the export lock path is unreachable on both platforms because export_hyper deletes the target before attaching, and suggested the live Windows issue may instead be the delete step's error classification. Whoever picks this up should read that analysis, since it covers adjacent code.
What is needed
A Windows runner. This could not be reproduced locally — all recent investigation has been on macOS, and the failure has not recurred in CI since.
Suggested first steps:
- Re-run the Windows leg repeatedly on an unchanged tree to establish a rate.
- Inspect
attach_tests.rs:312to determine whether the assertion depends on file-handle release timing after the first attach.
- Dominant language
- Rust
- Stars
- 2
- Forks
- 2
- Avg merge
- 12h 2m
- Merged PRs (30d)
- 60
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from tableau/hyper-api-rust
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
tableau/hyper-api-rust#294 ·
-
Difficulty 4/5 3-5 days Newbie friendliness 35/100
tableau/hyper-api-rust#311 ·
-
Difficulty 4/5 3-5 days Newbie friendliness 45/100
tableau/hyper-api-rust#305 ·
-
Windows Named Pipe: verify DACL denies other users, and measure read-path perf for MCP workloads Open
Difficulty 4/5 3-5 days Newbie friendliness 38/100
tableau/hyper-api-rust#302 ·
-
Difficulty 3/5 1-2 days Newbie friendliness 72/100
tableau/hyper-api-rust#300 ·
All issues in tableau/hyper-api-rust
Similar issues
-
risk:low runtime status:in-progress type:test
Difficulty 1/5 Under an hour Newbie friendliness 92/100
zeroclaw-labs/zeroclaw#11023 ·
-
good first issue refactor
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
EricSpencer00/Resilient#4835 · 1 comment ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
bisq-network/bisq-musig#204 ·
-
agent:ready documentation
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
cesarferreira/stax#890 ·