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

HHH-16233 Fix logs for mutable non-root entity #6169

Merged
merged 1 commit into from
Feb 27, 2023
Merged

HHH-16233 Fix logs for mutable non-root entity #6169

merged 1 commit into from
Feb 27, 2023

Conversation

shin-mallang
Copy link
Contributor

@shin-mallang shin-mallang commented Feb 27, 2023

An incorrect error log message is given about @immutable is declared to non-root entities that are not @immutable declared.

@immutable can only be declared on the Root Entity.
That is, it is not wrong to say that a non-root entity is mutable, so should not send the message @Immutable used on a non root entity to non-root entity that are not declared with @immutable.

This should only happen for non-root entities that are immutable.

https://hibernate.atlassian.net/jira/software/c/projects/HHH/issues/HHH-16233

@hibernate-github-bot
Copy link

hibernate-github-bot bot commented Feb 27, 2023

Thanks for your pull request!

This pull request appears to follow the contribution rules.

› This message was automatically generated.

@shin-mallang shin-mallang changed the title Fix logs for mutable non-root entity HHH-16233 Fix logs for mutable non-root entity Feb 27, 2023
@gavinking
Copy link
Member

Oops, I suck!

@gavinking gavinking self-assigned this Feb 27, 2023
@shin-mallang
Copy link
Contributor Author

@gavinking Did I do a good job??

@gavinking
Copy link
Member

@gavinking Did I do a good job??

Yes, sure, we just need to make sure the tests pass.

@shin-mallang
Copy link
Contributor Author

Oh, great! I'm glad I can contribute to Hibernate😊
thank you🙇🏻‍♂️

@gavinking gavinking merged commit 3edb588 into hibernate:main Feb 27, 2023
@gavinking
Copy link
Member

Merged, thanks!

@shin-mallang
Copy link
Contributor Author

I'm honored😊

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