Skip to content

Conversation

@beikov
Copy link
Member

@beikov beikov commented Jan 3, 2025


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license
and can be relicensed under the terms of the LGPL v2.1 license in the future at the maintainers' discretion.
For more information on licensing, please check here.


https://hibernate.atlassian.net/browse/HHH-4160

@gavinking
Copy link
Member

I don't understand why this test passes. We do accept detached entities in remove().

@gavinking
Copy link
Member

I don't understand why this test passes. We do accept detached entities in remove().

Oh, perhaps because use of @Jpa puts us in JPA-compliant mode.

@beikov
Copy link
Member Author

beikov commented Jan 3, 2025

I don't understand why this test passes. We do accept detached entities in remove().

Oh, perhaps because use of @Jpa puts us in JPA-compliant mode.

Correct, this is explicitly about JPA compliance as you can also read in the title of the PR.

@gavinking
Copy link
Member

OK, cool, that wasn't clear from the discussion on the issue.

@beikov beikov merged commit 3d60bfa into hibernate:main Jan 3, 2025
25 checks passed
@beikov beikov deleted the HHH-4160 branch January 3, 2025 14:27
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.

2 participants