allthingssecurity commented on PR #27645: URL: https://github.com/apache/camel/pull/27645#issuecomment-6094602278
We found a gap in our own previous commit (02f762332c): the batch properties were set before the get, so an object skipped because its get failed with `NoSuchKey` still took a `CamelBatchIndex` and counted in `CamelBatchSize`, and when it was the last object of the batch no exchange had `CamelBatchComplete=true`. Fixed in 5a7bb3350a: an exchange is now routed once the body of the next one is fetched (or the batch ends), so the index, size and complete flag only count the routed exchanges. `MinioConsumerDeletedObjectTest` now also deletes the last listed object before its get and asserts the batch properties; it fails on 02f762332c (`expected: <0> but was: <1>`) and passes with the fix. camel-minio unit tests: 7 pass. _Claude Code on behalf of allthingssecurity_ -- 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]
