Fix deprecation for the MappingJacksonParameterizedConverter

[artembilan]

The MappingJackson2MessageConverter is deprecated for removal in Spring Framework 7.0 in favor of JacksonJsonMessageConverter.

Related to: spring-projects/spring-framework#33798

[chickenchickenlove]

Hi, @artembilan
Would it be okay for me to handle this issue?

[sobychacko]

@chickenchickenlove I say go for it.

[artembilan]

Not yet.
See related Spring Framework issue: spring-projects/spring-framework#34963

[chickenchickenlove]

roger that.
I've got it!

[chickenchickenlove]

@artembilan
I have a question.

Would it be okay to create PR for spring-projects/spring-framework#34963?
I’m not sure whether you are waiting for the framework team’s work or if it’s acceptable to work on another contributor’s PRs.

If it’s the latter, would it be alright for me to open a PR?

[artembilan]

Thank you for offer, @chickenchickenlove !
Yes, you can issue PR in the Framework project.
That might expedite feedback loop.
However, we are not in rush for this issue.
You cannot find anything else to work on? 😉

[artembilan]

FYI, the required issue in the Framework has been fixed.
Feel free to contribute this migration to a new API!
Thanks

[artembilan]

There was misunderstanding from my side.
The new JacksonJsonMessageConverter in Spring Framework is for Jackson 3.
While it is recommended for end-users to migrate to Jackson 3 and that is how we got an encourage from that deprecated MappingJackson2MessageConverter, but that does not mean that we have a freedom in library like this ours Spring for Apache Kafka and make decision for consumers of it.
Therefore we have to go here slightly different direction:

1. Manage Jackson 3 dependency.
2. Implement a MappingJacksonParameterizedConverter variant for that Jackson 3 based JacksonJsonMessageConverter
3. And deprecated existing for Jackson 2.

This way, even if we phasing out Jackson 2 support, that does not mean that we drop it off altogether.
After that we would need to think about all the places where we use Jackson 2 directly, e.g. DefaultKafkaHeaderMapper and Json(De)Serializer.
Looks like Spring Framework has Jackson 3 variants for all the possible converters and encoders, e.g. new AbstractJacksonHttpMessageConverter.

[sobychacko]

Ya, eventually, I think we should migrate all our Jackson support to the new one.

[sobychacko]

@chickenchickenlove We can look at this issue since we need to tackle a bunch of external dependencies, deprecations, etc., and then coordinate with the framework changes. Were you already in the process of making any changes?

[chickenchickenlove]

@sobychacko
Sorry to late 🙇‍♂
I just took a look code about how to deal with this problem and I didn't start making any changes with code.
So, If you want to take this task, please go ahead.
Sorry for inconvenience. 🙇‍♂