-
Notifications
You must be signed in to change notification settings - Fork 40.2k
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
Upgrade to Spring Kafka 3.0.0-M5 #31620
Comments
The latest changes break retries due to there being no bean named 'org.springframework.kafka.retrytopic.internalRetryTopicConfigurer'. We have this code at the moment: Lines 148 to 174 in fbba3ff
I think we may need to auto-configure a @garyrussell @tomazfernandes can you please help us here? |
Hi @wilkinsona, makes sense, sorry we missed that.
Perhaps @garyrussell and @artembilan might have a different idea, but this seems like the proper behavior to me. But forgive my ignorance - why would we need a sub-class instead of the class itself?
As long as we're able to autoconfigure the support bean, I don't think this should change anything - all the autoconfiguration does so far is add a I'm not sure if that's something you or @garyrussell would like to fix yourselves - once we figure out the proper behavior I'm available to write a PR if needed. Thanks. |
I've yet to grok the difference between a |
The The Of course, perhaps the names could be clearer on its intentions, and probably the configuration strategy as a whole has room for improvement as well. |
This will affect Boot 2.7 users that wish to use s-k 2.9 as well. Need to rethink. |
No description provided.
The text was updated successfully, but these errors were encountered: