Skip to content

Port Hibernate's lazy attribute fetch groups #1949

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

Merged
merged 4 commits into from
Feb 14, 2019

Conversation

maca88
Copy link
Contributor

@maca88 maca88 commented Dec 28, 2018

This PR ports:
HHH-10267 - Support defining lazy attribute fetch groups
HHH-11506 - Fix update of lazy attributes when there are attributes not loaded

Additionally, this PR updates the cache after a lazy group is initialized to be consistent with #1922.

Should be reviewed after #1922.

@hazzik

This comment has been minimized.

@maca88

This comment has been minimized.

@maca88
Copy link
Contributor Author

maca88 commented Feb 1, 2019

Rebased.

@maca88 maca88 changed the title WIP - Port Hibernate's lazy attribute fetch groups Port Hibernate's lazy attribute fetch groups Feb 1, 2019
@fredericDelaporte

This comment has been minimized.

And do a renaming
@fredericDelaporte fredericDelaporte added this to the 5.3 milestone Feb 10, 2019
Copy link
Member

@fredericDelaporte fredericDelaporte left a comment

Choose a reason for hiding this comment

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

LGTM.

Has anyone checked how I have documented it?

@hazzik
Copy link
Member

hazzik commented Feb 12, 2019

Yes

@maca88
Copy link
Contributor Author

maca88 commented Feb 12, 2019

I've also checked, thanks @fredericDelaporte.

@fredericDelaporte fredericDelaporte merged commit 6cb478f into nhibernate:master Feb 14, 2019
@maca88 maca88 deleted the LazyGroup branch September 27, 2019 20:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants