Skip to content

Interfering Jackson annotations #512

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

Closed
nishisan opened this issue Jul 25, 2023 · 2 comments · Fixed by #513
Closed

Interfering Jackson annotations #512

nishisan opened this issue Jul 25, 2023 · 2 comments · Fixed by #513
Assignees
Labels

Comments

@nishisan
Copy link

Hello, we recently upgraded the drivers from version 6 to version 7. After the upgrade, the replacements for @DocumentField(DocumentField.Type.KEY) and @DocumentField(DocumentField.Type.ID) introduced new annotations: @InternalKey and @InternalId.

The problem we encountered is that these new annotations come with a @JsonProperty, which replaces the original property name of the class, thereby breaking existing integrations prior to version 7. The workaround we found was to add an extra @JsonProperty annotation to the property fields in each class.

@rashtao
Copy link
Collaborator

rashtao commented Jul 26, 2023

Thanks for reporting it. It will be addressed by #513 , when complete.
In the meantime you can work around it using different classes to carry data across different layers of your application, or by customizing the driver user-serde as in #513.

@rashtao rashtao self-assigned this Jul 26, 2023
@nishisan
Copy link
Author

That is a too expensive workarround, in mean time we rolled back to version 6. As we figured out that json serialization with spring api´s and arangodb´s java driver will break the existing api contracts.
Thansk for the feedback.

@rashtao rashtao changed the title Upgrading Guide for Driver from 6.xxx to 7.xxx Interfering Jackson annotations Sep 12, 2023
@rashtao rashtao added the bug label Sep 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants