Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Tangles #550

Closed
garyrussell opened this issue Feb 1, 2018 · 0 comments
Closed

Fix Tangles #550

garyrussell opened this issue Feb 1, 2018 · 0 comments
Assignees
Labels
Milestone

Comments

@garyrussell
Copy link
Contributor

garyrussell commented Feb 1, 2018

listener <-> config

Move AbstractMessageListenerContainer.AckMode to config or support.

Move error handler interfaces to support.

HandlerAdapter

screen shot 2018-02-01 at 9 45 18 am

@garyrussell garyrussell added this to the 2.2.M1 milestone Feb 1, 2018
@garyrussell garyrussell self-assigned this Apr 3, 2018
garyrussell added a commit to garyrussell/spring-kafka that referenced this issue Apr 3, 2018
Fixes spring-projects#550

- package tangle `....listener` and `....listener.config` - remove config package
- class tangles between `ContainerProperties` and the listener containers
 - AckMode moved to properties
 - Error handler setters moved from properties to containers
garyrussell added a commit to garyrussell/spring-kafka that referenced this issue Apr 3, 2018
Fixes spring-projects#550

- package tangle `....listener` and `....listener.config` - remove config package
- class tangles between `ContainerProperties` and the listener containers
  - AckMode moved to properties
  - Error handler setters moved from properties to containers
denis554 added a commit to denis554/spring-kafka that referenced this issue Mar 27, 2019
Fixes spring-projects/spring-kafka#550

- package tangle `....listener` and `....listener.config` - remove config package
- class tangles between `ContainerProperties` and the listener containers
  - AckMode moved to properties
  - Error handler setters moved from properties to containers
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant