Skip to content

Fix caching linq query with ThenFetchMany #2558

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 5 commits into from
Sep 23, 2020

Conversation

illkostinf
Copy link
Contributor

@illkostinf illkostinf commented Sep 21, 2020

The commit contains an unit test for #2559 that performs caching linq query with using ThenFetchMany statement and tries to retrieve cached query result set. Starting from nhibernate 5.3 retrieving such cached query result set is not working so unit test is failing. Actually an unit test shows the bug in nhibernate 5.3.

@fredericDelaporte
Copy link
Member

fredericDelaporte commented Sep 21, 2020

Do you mean this was working with 5.2.x?

@illkostinf
Copy link
Contributor Author

Do you mean this was working with 5.2.x?

Yes, it was working with 5.2.7

@maca88 maca88 force-pushed the CachingQueryWithThenFetchMany branch from 38a0e2a to 9e1f930 Compare September 22, 2020 11:23
@maca88 maca88 changed the base branch from master to 5.3.x September 22, 2020 11:24
@maca88
Copy link
Contributor

maca88 commented Sep 22, 2020

Rebased to 5.3.x and added a fix.

@maca88 maca88 force-pushed the CachingQueryWithThenFetchMany branch from 08913f1 to e6662d6 Compare September 22, 2020 11:32
@fredericDelaporte fredericDelaporte added this to the 5.3.4 milestone Sep 22, 2020
@fredericDelaporte fredericDelaporte changed the title Added unit test that checks caching query result set with ThenFetchMa… Fix caching linq query with ThenFetchMany Sep 22, 2020
@hazzik hazzik merged commit da95db4 into nhibernate:5.3.x Sep 23, 2020
bahusoid pushed a commit to bahusoid/nhibernate-core that referenced this pull request Aug 10, 2021
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.

5 participants