Skip to content

Conversation

@sobychacko
Copy link
Contributor

…ontainerFactory

Enables configuration of container properties at the factory level, following the established pattern in other Spring Kafka container factories. This provides a more flexible and Spring-friendly way to configure share consumer containers, particularly for settings like explicit acknowledgment mode, without requiring configuration through Kafka client properties.

  • Add getContainerProperties() method to allow configuration at factory level
  • Copy factory-level properties to each listener container instance during creation
  • Update acknowledgment mode determination to respect factory-level settings with proper precedence
  • Add integration test for factory-level explicit acknowledgment configuration
  • Update documentation with factory-level configuration example

@sobychacko sobychacko added this to the 4.0.0-RC1 milestone Oct 17, 2025
…ontainerFactory

Enables configuration of container properties at the factory level, following
the established pattern in other Spring Kafka container factories. This provides
a more flexible and Spring-friendly way to configure share consumer containers,
particularly for settings like explicit acknowledgment mode, without requiring
configuration through Kafka client properties.

- Add `getContainerProperties()` method to allow configuration at factory level
- Copy factory-level properties to each listener container instance during creation
- Update acknowledgment mode determination to respect factory-level settings with proper precedence
- Add integration test for factory-level explicit acknowledgment configuration
- Update documentation with factory-level configuration example

Signed-off-by: Soby Chacko <soby.chacko@broadcom.com>
@sobychacko sobychacko force-pushed the share-consumer-improvements branch from a0a3a3c to 5fc8220 Compare October 17, 2025 20:08
@artembilan artembilan merged commit bedf408 into spring-projects:main Oct 17, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants