Yicong-Huang opened a new pull request, #4527: URL: https://github.com/apache/texera/pull/4527
> [!IMPORTANT] > Draft / experiment. Goal is to use CI to find out whether the test suite is still stable under sbt's default parallelization. If CI is green across multiple runs, this becomes a real PR. If something flakes, we identify the specific shared state instead of carrying the global workaround. ### What changes were proposed in this PR? Removed the line ```sbt Global / concurrentRestrictions += Tags.limit(Tags.Test, 1) ``` from all 11 module `build.sbt` files where it was copy-pasted (`amber`, `common/auth`, `common/config`, `common/dao`, `common/pybuilder`, `common/workflow-core`, `common/workflow-operator`, `access-control-service`, `file-service`, `config-service`, `workflow-compiling-service`). This restriction forces every test in every module to run sequentially in a single JVM, and is one of the biggest single contributors to backend test runtime in CI. ### History `git log -S "concurrentRestrictions in Global"` traces the line to commit `6a79a655ca` (2020-12-28), part of PR #938 (*Extending the runtime error raising framework to all actors*). That PR mainly reworked exception handling across `Controller`, `Principal`, and `WorkerBase`. The restriction was added in the same PR with the only comment being "ensuring no parallel execution of multiple tasks" — no specific reason given. Most likely the actor-error refactor introduced shared mutable state that couldn't tolerate parallel test JVMs and the global `Tags.limit` was applied as a quick fix, then copy-pasted into every new module. ### Any related issues, documentation, discussions? Refs #4525. ### How was this PR tested? Local full-suite run is impractical (JDK 17 + Arrow incompatibility blocks several e2e specs). Relying on this PR's CI on JDK 11 to validate. Expected to need 2–3 reruns to confirm stability, since flakiness from shared state may be intermittent. ### Was this PR authored or co-authored using generative AI tooling? Generated-by: Claude Code (Opus 4.7) -- 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]
