ryerraguntla commented on code in PR #4043: URL: https://github.com/apache/iggy/pull/4043#discussion_r4001631127
########## .config/nextest.toml: ########## @@ -39,6 +39,23 @@ max-threads = 1 filter = 'package(integration) and test(/connectors::elasticsearch::/)' test-group = "elasticsearch" +# Each test in this binary spawns its own real iggy-server subprocess (isolated data dir, port +# from PortGuard's own flock, so no two ever share a port). #[serial] on them is a dead letter +# under nextest - nextest runs every test as its own process, and #[serial]'s mutex only excludes +# within one process (nextest does not support #[serial] macro, same gap the workaround above +# documents). Without this override, nextest's default scheduler runs several of these processes +# at once, and every one of them inherits this workspace's cpu_allocation = "numa:auto" / Review Comment: **Fixed** - comment now justifies max-threads = 1 by the numa:auto contention alone, dropped the stale pin-cores reasoning. Kept the override itself, -- 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]
