Skip to content

Conversation

dreab8
Copy link
Member

@dreab8 dreab8 commented Sep 16, 2025

Changes needed to fix hibernate/hibernate-reactive#1770
see hibernate/hibernate-reactive#2513


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

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: [04a6c1b]

› This message was automatically generated.

@beikov
Copy link
Member

beikov commented Sep 22, 2025

Since you're overriding AbstractEntityPersister, wouldn't it be better/easier to just throw a better error from your subclass in the initializeEnhancedEntityUsedAsProxy method?

@dreab8
Copy link
Member Author

dreab8 commented Sep 23, 2025

Hi @beikov , Reactive Stage#fetch() method uses initializeEnhancedEntityUsedAsProxy to initialize the entity attributes.

In Reactive we have to prevent calling this method directly from the LazyAttributeLoadingInterceptor and EnhancementAsProxyLazinessInterceptor so what you suggest unfortunately doesn't work.

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.

Check error message with lazy loading fields
2 participants