-
-
Notifications
You must be signed in to change notification settings - Fork 409
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
[TECH-DEBT] Move to jackson json provider in JsonPath instead of jsonsmart #648
Comments
@a1shadows You may assign this issue to me. I'll have a closer look at json array handling :-) |
@a1shadows Do you know the reason, why Gson is used in the Kafka part? Couldn't that also be changed to Jackson? |
Hey @authorjapps any idea on this? |
@a1shadows, I vaguely remember, a collaborator introduced for a quick fix. I think it was introduced for Kafka "headers" (and elsewhere) to allow generic object structure ser/de using Gson easily over Jackson. Worth checking anywhere else Gson is not used before completely getting rid of it. |
We can have uniformity across the repo if we do this and can avoid pulling in the json-smart dependency
was raised during #647
The text was updated successfully, but these errors were encountered: