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-18229 Handle null owner key for collections #9700

Open
wants to merge 1 commit into
base: 6.2
Choose a base branch
from

Conversation

soul2zimate
Copy link

(cherry picked from commit 1f08501)

upstream PR: #8700

issue: https://hibernate.atlassian.net/browse/HHH-18229

[Backport HHH-18229 fix to downstream 6.2 branch]


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.


@mbladel
Copy link
Member

mbladel commented Feb 6, 2025

@soul2zimate it looks like compilation is failing, could you please check?

@soul2zimate
Copy link
Author

@soul2zimate it looks like compilation is failing, could you please check?

sure, missed the Nullable import statement for some classes. I have updated the PR.

@mbladel
Copy link
Member

mbladel commented Feb 7, 2025

@soul2zimate looks like the build is still failing, are you going to look into it?

@soul2zimate
Copy link
Author

soul2zimate commented Feb 7, 2025

@soul2zimate looks like the build is still failing, are you going to look into it?

sorry @mbladel I am on an unexpected PTO and I don't have my laptop nearby. There is no compilation error but it seems there are failed tests which I don't understand why. I can only investigate them next Monday.

@mbladel
Copy link
Member

mbladel commented Feb 7, 2025

No problem, just let us know if you need help or if you won't be able to look into it yourself we can always give it a try.

Copy link
Author

@soul2zimate soul2zimate left a comment

Choose a reason for hiding this comment

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

Hi @mbladel, The Tests / Build/ ... checks are passed. Can you check if this is good for you ?

@mbladel mbladel added the 6.2 label Feb 10, 2025
Copy link
Member

@mbladel mbladel left a comment

Choose a reason for hiding this comment

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

Looks great now, thanks @soul2zimate!

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 this pull request may close these issues.

2 participants