Skip to content

Resolve deprecations in spring-integration-kafka after upgrade to Spring for Apache Kafka 2.8/30 #3661

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

Closed
artembilan opened this issue Nov 3, 2021 · 3 comments · Fixed by #3751

Comments

@artembilan
Copy link
Member

No description provided.

@artembilan
Copy link
Member Author

Need also to figure out what to do with already deprecated org.springframework.kafka.listener.adapter.RetryingMessageListenerAdapter in the KafkaMessageDrivenChannelAdapter

@50qbits
Copy link

50qbits commented Mar 16, 2022

Hello, I've found that KafkaMessageListenerContainerSpec.errorHandler is using GenericErrorHandler and this is deprecated in favor of DefaultErrorHandler (since spring-kafka 2.8).

Besides spring-kafka ConcurrentKafkaListenerContainerFactory and KafkaMessageListenerContainer.setErrorHandler​(ErrorHandler errorHandler) is deprecated in favor of setCommonErrorHandler(CommonErrorHandler)

Any plans to updated this before 6.0.x?

@garyrussell
Copy link
Contributor

I have almost completed the PR. It should be posted today and will be in the next milestone.

@garyrussell garyrussell modified the milestones: 6.0.x, 6.0 M2 Mar 16, 2022
garyrussell added a commit to garyrussell/spring-integration that referenced this issue Mar 16, 2022
garyrussell added a commit to garyrussell/spring-integration that referenced this issue Mar 16, 2022
garyrussell added a commit to garyrussell/spring-integration that referenced this issue Mar 16, 2022
artembilan pushed a commit that referenced this issue Mar 16, 2022
Resolves #3661

* Don't allow existing listener; remove unnecessary try/catch; add what's new.
* Fix link in whats-new.adoc
garyrussell added a commit to garyrussell/spring-integration that referenced this issue Sep 21, 2022
Use of the `RetryingMessageListenerAdapter` was removed in
spring-projects@0550380
but the javadocs were not corrected.
artembilan pushed a commit that referenced this issue Sep 21, 2022
Use of the `RetryingMessageListenerAdapter` was removed in
0550380
but the javadocs were not corrected.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants