davsclaus commented on PR #24863: URL: https://github.com/apache/camel/pull/24863#issuecomment-5005720889
Both issues are already addressed in the latest commit on this branch: 1. **ShardIteratorHandler conflict**: The branch was rebased on main after PR #24861 merged. The overlapping ShardIteratorHandler changes were dropped — `git diff origin/main..fix/CAMEL-24156` shows no ShardIteratorHandler hunks. 2. **KinesisResumeAction registry lookup**: Restored in commit `3c88130`. The `resolveResumeAction()` method now looks up a user-registered action from the registry via `lookupByNameAndType()`. If found, it creates a per-shard instance via the no-arg constructor of the template's class (preserving custom subclass behavior), then sets the builder. This fixes the thread-safety issue while preserving backward compatibility. The CI run triggered by the earlier push should validate both fixes. No force-push needed — the branch is clean against current main. _Claude Code on behalf of Claus Ibsen_ -- 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]
