-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Optionally configure serialisers #2274
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
Comments
I agree that we should add a flag to disable the In the meantime, as a work around, use
It's probably a bug, but setting the serializer this way skips the |
Yes, it really looks like a bug. According to that your PR: https://github.com/spring-projects/spring-kafka/pull/1907/files#diff-ee2dfb7123c02f066d237b437f06f37c6ab26279ed47f3edd8b7ca0d0a4c96cdR285, the But your idea to leverage a setter API for this kind of requested option indeed makes sense for me. |
Resolves spring-projects#2274 Provide an option to suppress configuring programmatically provided `Serializer`s and `Deserializer`s. Also fix the DKPF, which did not configure serializers provided by the supplier setter.
Resolves #2274 Provide an option to suppress configuring programmatically provided `Serializer`s and `Deserializer`s. Also fix the DKPF, which did not configure serializers provided by the supplier setter.
Resolves #2274 Provide an option to suppress configuring programmatically provided `Serializer`s and `Deserializer`s. Also fix the DKPF, which did not configure serializers provided by the supplier setter.
Expected Behavior
Sometimes you have already configured your serialiser by calling
configure()
on it and do not want the spring framework to reconfigure it.Current Behavior
In version 2.8.4, if you have already configured your serialiser, the DefaultKafkaProducerFactory attempts to reconfigure it from scratch and all your settings are lost.
Context
It would be nice to have a flag/switch to indicate whether you want spring to configure your serialiser for you or not as discussed in this Stackoverflow thread: https://stackoverflow.com/questions/72228636/spring-kafka-serialiser-config/72231032?noredirect=1
The text was updated successfully, but these errors were encountered: