jayakasadev opened a new pull request, #3578:
URL: https://github.com/apache/iggy/pull/3578
## Which issue does this PR address?
Relates to #3315
## Rationale
* decoupling `core/server-ng` from the legacy `core/server` crate
## What changed?
* allocator moved to `server_common`, which is used by both crates already
* `server-ng` had to import `server` crate to use shard allocator
(`server::shard_allocator`) before this
* `CpuAllocation`/`NumaConfig` moved to `server_common`
* needed by allocator and `configs`
* `configs::sharding` re-exports them to avoid a `server_common` ->
`configs` -> `server_common` cycle
## Local Execution
- Passed (macos); Linux/musl builds left to CI
- Pre-commit hooks ran
## AI Usage
1. Tools: Cursor (Claude)
2. Scope: planned and performed the full relocation
3. Verification:
* ran fmt/sort/clippy + unit tests locally
* confirmed `rg 'server::shard_allocator' core/server-ng/` is empty and
that nothing was broken by the move
4. Yes, I can explain every line
--
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]
