Skip to content

Conversation

@gavinking
Copy link
Member

EntityMetamodel has been deprecated for removal since 6.0.


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.


@hibernate-github-bot
Copy link

hibernate-github-bot bot commented Sep 2, 2025

Thanks for your pull request!

This pull request does not follow the contribution rules. Could you have a look?

❌ All commit messages should start with a JIRA issue key matching pattern HHH-\d+
    ↳ Offending commits: [b02eabf, 8fa81aa, 1dfba53, 6be5d50, ee55d2e, c28c1ef, 3867983, a795f3b, 563093a]

› This message was automatically generated.

Comment on lines +266 to +267
final boolean lazy =
isLazy( creationContext, property, collectionsInDefaultFetchGroup, bytecodeEnhancementMetadata );

Check notice

Code scanning / CodeQL

Possible confusion of local and field Note

Confusing name:
EntityMetamodel
also refers to field
lazy
(without qualifying it with 'this').
@gavinking gavinking marked this pull request as ready for review September 2, 2025 04:43
@gavinking
Copy link
Member Author

I guess I'm going to go ahead and merge these changes, even though @sebersole has not yet given his opinion, because otherwise I'm just going to get a mess of conflicts with other work.

The two very important commits here are easy enough to roll back of Steve hates them or has a better idea. (Most of the commits on this PR are included here only to avoid conflicts.)

@gavinking gavinking merged commit d95e14a into hibernate:main Sep 3, 2025
23 of 26 checks passed
beikov added a commit to beikov/hibernate-orm that referenced this pull request Sep 10, 2025
beikov added a commit that referenced this pull request Sep 10, 2025
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.

1 participant