Jackie-Jiang opened a new pull request, #19893: URL: https://github.com/apache/pinot/pull/19893
Follow-up to #19737 addressing the remaining review suggestions. Queued repairs previously retained metadata from the request, so a segment deleted or repaired by another replica could still start re-ingestion later. Re-resolve the table manager and fetch current segment metadata when the worker starts, skipping unavailable tables and segments that are missing, no longer COMMITTING, or already have a download URL. Skipped jobs are removed without incrementing the failure metric. This rechecks queued work; it does not coordinate repairs already running on different replicas. - Use one segment-keyed job map for duplicate prevention and job listing, with conditional removal on completion or rejected submission. - Remove the injection-only API test and consolidate overlapping timeout tests while retaining cluster-only configuration coverage. - Add queued-job regression cases for deleted, completed, non-committing, and already-uploaded segments. Validation: all 30 tests in `ReingestionResourceApiTest`, `ReingestionResourceTest`, and `ReingestionConsumptionTimeoutTest` passed using the `pinot-server` reactor build. Spotless, Checkstyle, license formatting, and license checks passed. Compilation ran with deprecation and warning reporting enabled. -- 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]
