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

JsonInclude.Include.NON_EMPTY affects dirty check #590

Open
firedes opened this issue Mar 14, 2023 · 3 comments
Open

JsonInclude.Include.NON_EMPTY affects dirty check #590

firedes opened this issue Mar 14, 2023 · 3 comments

Comments

@firedes
Copy link

firedes commented Mar 14, 2023

JsonInclude.Include.NON_EMPTY affects dirty check, even json mapping properties override equals and hashCode correctly.
Don't know why, didn't find any description about this in hypersistence-utils documents.

@vladmihalcea
Copy link
Owner

Try to create a replicating test case that proves the problem.

There are many tests asserting the number of statements that are executed, so you can use that mechanism to demonstrate how the dirty checking mechanism is affected.

@firedes
Copy link
Author

firedes commented Mar 14, 2023

@vladmihalcea Added a pr #591

@vladmihalcea
Copy link
Owner

Great. Now, the next step is to debug the library and see why it doesn't work. Let me know what you have found out so that we can see what's the best way to address the problem.

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

No branches or pull requests

2 participants