xiangfu0 opened a new pull request, #17843: URL: https://github.com/apache/pinot/pull/17843
## What This updates the integration test runner script used in GitHub Actions to make surefire fork startup failures easier to diagnose and reduce fork startup memory pressure for set 2. ### Changes - Add `print_surefire_dumps()` in `.github/workflows/scripts/pr-tests/.pinot_tests_integration.sh` - On integration-test failure, print surefire dump artifacts from `target/surefire-reports` (`*.dump`, `*.dumpstream`, `*jvmRun*`) - For `RUN_TEST_SET=2`, pass `-DargLine="-Xms1g -Xmx2g -Dlog4j2.configurationFile=log4j2.xml"` to lower fork JVM heap from default 4g/4g ## Why We saw CI fail with: - `ExecutionException Error occurred in starting fork` - `SurefireBooterForkException` The dump files are the primary source of root-cause details for this class of error. Lowering surefire fork heap for set 2 should also reduce startup failures due to memory pressure on GitHub runners. ## Validation - `bash -n .github/workflows/scripts/pr-tests/.pinot_tests_integration.sh` -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
