justinmclean opened a new pull request, #4243:
URL: https://github.com/apache/iggy/pull/4243

   ## Which issue does this PR address?
   
   Closes #4242
   
   ## Rationale
   
   The documented Docker quickstart does not start 0.9.0 on Docker Desktop or 
other single-NUMA-node hosts.
   
   ## What changed?
   
   With one NUMA node, every shard bound its memory to that node. The bind 
gains nothing there, and Docker Desktop's VM rejects it, so the server exited 
at startup with failed to bind shard N memory to its NUMA node.
   
   Shards are no longer memory-bound when the topology has a single node. 
Multi-node hosts are unchanged. NumaTopology::detect() is split so tests can 
pass synthetic one-node and two-node topologies.
   
   ## Local Execution
   
   - Passed: cargo test -p server --lib (460 of 460), cargo fmt --check, cargo 
clippy --all-features. The new single-node test fails without the fix.
   - Pre-commit hooks ran (prek).
   - Not run: an end-to-end check against a rebuilt Docker image. The 
workaround in #4242 was verified in Docker on 0.9.0.
   
   ## AI Usage
   
   1. Claude (Claude Code).
   2. Diagnosed the startup failure and wrote the fix, the from_topology split, 
and the two tests.
   3. Ran the server lib tests, fmt, and clippy, and confirmed the single-node 
test fails when the fix is reverted.
   4. Reviewed and refined by a human.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to