xiangfu0 opened a new pull request, #19196: URL: https://github.com/apache/pinot/pull/19196
## Summary - add an explicit serial TestNG suite for the nested Kinesis ingestion tests, which are not selected by the current top-level alphabetical includes - share one Pinot and LocalStack fixture across RealtimeKinesisIntegrationTest and KinesisShardChangeTest while keeping per-class tables, schemas, and streams isolated - make setup, publisher-thread handling, and teardown failure-safe so one scenario cannot leak state into the next - run the suite on existing test Set 2 before coverage upload; the workflow still has exactly two matrix jobs ## Validation - full Docker-backed suite: 23 tests, 0 failures, 0 errors, 0 skips; TestSuite 900.2s, Maven 15:09 - pinot-integration-tests test-compile passed - Spotless, Checkstyle, license format/check, and git diff check passed - TestNG XML, shell script, workflow YAML, and effective-POM profile resolution passed - independent lifecycle and isolation review found no remaining blocker ## Runtime note This draft intentionally restores missing CI coverage, so it adds work even after fixture sharing. The local run took about 15 minutes. Hosted CI should establish the actual Set 2 cost and whether a narrower smoke matrix is preferable before marking the PR ready. -- 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]
