Skip to content
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

[7.67.x] [FIX] custom fix when xml adapter has names #3059

Open
wants to merge 1 commit into
base: 7.67.x
Choose a base branch
from

Conversation

github-actions[bot]
Copy link

@github-actions github-actions bot commented Aug 1, 2024

Backport: #3052

Note: CI is not automatically triggered on backported PRs, please comment 'ok to test' to launch Jenkins jobs

The performance problem was related to the polymorphic feature of jackson. it computes the subtypes of a given types.
Before the patch we were saying that all classes contained in the kjar/project where subtypes of every class causing a huge impact in performance
That performance problem was causing another problem, it was overwriting the proper names for types
Every class when you are serializing you have the full qualified name and the name used for serialized and deserialized
So before the patch we were overwriting that information making this feature not working properly
Now because the subtypes are being computed properly we are not overwriting the names therefore now the name is being taken from the annotation like it should be

@elguardian
Copy link
Member

ok to test

@elguardian
Copy link
Member

this ok to test

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant