arunkumarucet opened a new pull request, #17512: URL: https://github.com/apache/pinot/pull/17512
## Overview This PR introduces a storage quota check for each segment during batch segment upload in `PinotSegmentUploadDownloadRestletResource`. The change ensures that every segment in a batch is validated against the table's storage quota before being committed. ## Motivation Previously, batch uploads did not enforce storage quota checks, which lead to quota violations. This change aligns batch upload behavior similar to single segment uploads, improving quota enforcement and cluster stability. ## Implementation Added a call to `SegmentValidationUtils.checkStorageQuota()` for each segment in the batch upload loop. If any segment fails the quota check, a ControllerApplicationException is thrown, aborting the batch operation. No segments are committed if any segment fails validation; the operation is atomic at the batch level. Temporary files for all processed segments are cleaned up in the finally block, ensuring no resource leaks. -- 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]
