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

Add support for lazy loaded attribute fetching with JPA entity graph query hint #387

Merged
merged 3 commits into from Nov 22, 2023

Conversation

igdianov
Copy link
Collaborator

To add query support for entities with lazy attributes compiled using Hibernate bytecode enhancement plugin.

@igdianov igdianov self-assigned this Nov 22, 2023
@igdianov igdianov changed the title Add support for lazy loaded attribute fetching with JPA entity graph hint Add support for lazy loaded attribute fetching with JPA entity graph query hint Nov 22, 2023
Copy link

codecov bot commented Nov 22, 2023

Codecov Report

Attention: 7 lines in your changes are missing coverage. Please review.

Comparison is base (17120b6) 75.54% compared to head (da61133) 75.73%.

Files Patch % Lines
.../jpa/query/schema/impl/GraphQLJpaQueryFactory.java 91.66% 0 Missing and 5 partials ⚠️
...ventures/graphql/jpa/query/schema/JavaScalars.java 33.33% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master     #387      +/-   ##
============================================
+ Coverage     75.54%   75.73%   +0.19%     
- Complexity     1150     1175      +25     
============================================
  Files            76       76              
  Lines          5181     5239      +58     
  Branches        739      747       +8     
============================================
+ Hits           3914     3968      +54     
  Misses          940      940              
- Partials        327      331       +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@igdianov igdianov force-pushed the add-lazy-load-attributes-fetchgraph branch from 64798bf to cdebc1b Compare November 22, 2023 00:53
@igdianov igdianov force-pushed the add-lazy-load-attributes-fetchgraph branch from cdebc1b to f9e489a Compare November 22, 2023 00:56
@igdianov igdianov closed this Nov 22, 2023
@igdianov igdianov reopened this Nov 22, 2023
@igdianov igdianov force-pushed the add-lazy-load-attributes-fetchgraph branch from df56453 to 6df9674 Compare November 22, 2023 19:12
@igdianov igdianov force-pushed the add-lazy-load-attributes-fetchgraph branch from 6df9674 to da61133 Compare November 22, 2023 19:21
@igdianov igdianov merged commit 715021d into master Nov 22, 2023
4 checks passed
@igdianov igdianov deleted the add-lazy-load-attributes-fetchgraph branch November 22, 2023 20:17
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.

None yet

1 participant