Jackie-Jiang commented on PR #20106: URL: https://github.com/apache/pinot/pull/20106#issuecomment-6699832979
@deepthi912 Addressed both requests from [your comment](https://github.com/apache/pinot/pull/20106#issuecomment-6671651959) in fdf7ca103b. - Added a real rebalance regression with `salary: INCREMENT`. The destination is already consuming another partition of the same table. A test metadata manager delays the older base segment S1 while S2 finishes loading; the test checks that S3 waits with an unchanged offset and no indexed rows. It pushes increments of 7 and 11, releases S1, and requires a completed rebalance and salary `100 + 7 + 11 = 118` (table total `220`). All metadata operations delegate to real upsert managers. - Added a TODO at the startup readiness check for coordinating RESET/manual deletion with an already running consumer, and clarified that boundary in the PR description. Compilation and formatting, checkstyle, and license checks passed. No tests were executed locally, as requested; the new regression awaits CI. Also merged current master and resolved the import conflict. -- 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]
