Skip to content

Conversation

@cigaly
Copy link
Contributor

@cigaly cigaly commented Jun 16, 2025

Jira issue HHH-19549

Always use access type specified when @Access annotation is present on embeddable.


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.


@cigaly cigaly force-pushed the HHH-19549-Enclosing_element_must_not_change_access_type_of_embeddable_annotated_with_@Access branch 2 times, most recently from 854098d to 992ddc8 Compare July 15, 2025 14:01
Copy link
Member

@gavinking gavinking left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cigaly I must apologize for taking so long to review this.

This fix looks good to me.

But since I took so long to review this, it looks like changes to the testing framework have broken your test. This is completely my fault.

If you can fix the test, I promise I will merge the PR immediately.

Really sorry about that.

@cigaly cigaly force-pushed the HHH-19549-Enclosing_element_must_not_change_access_type_of_embeddable_annotated_with_@Access branch from 992ddc8 to 6cbbf38 Compare July 19, 2025 18:45
@gavinking gavinking merged commit 97a0c68 into hibernate:main Jul 19, 2025
23 of 24 checks passed
@gavinking
Copy link
Member

Merged! Thanks mate

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants