allthingssecurity commented on PR #27645: URL: https://github.com/apache/camel/pull/27645#issuecomment-6094929311
@davsclaus thanks, agreed. Reverted 5a7bb335 in e33ee2069e: the batch properties are set up front again, before the get, as in 02f76233. Kept 02f76233 (a `NoSuchKey` on the stat or get releases and skips the object) and 82243080 (`processBatch` returns `total - skipped`; the revert keeps the skipped counter). bd2fa5fae3 adds a NOTE under "Batch Consumer" in `minio-component.adoc` and the catalog copy: if an object is deleted between the listing and the get, it is skipped, so `CamelBatchSize` can be higher than the number of routed exchanges and possibly no exchange has `CamelBatchComplete=true`. `MinioConsumerDeletedObjectTest` drops the batch-property assertions and now checks, with a `pollStrategy`, that the skipped object does not count as polled (fails with `expected: <1> but was: <2>` when it does). camel-minio unit tests: 7 run, 0 failures (the MinIO ITs are disabled on main). PR description updated. _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]
