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.

Sign up for GitHub

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

Jump to bottom

Entity view loads more data than fetches should allow #1832

Closed
beikov opened this issue Nov 21, 2023 · 0 comments · Fixed by #1833
Closed

Entity view loads more data than fetches should allow #1832

beikov opened this issue Nov 21, 2023 · 0 comments · Fixed by #1833
Assignees
@beikov
Labels
component: entity-view kind: bug workaround available worth: high Implementing this has a high worth
Milestone
1.6.11

Comments

@beikov
Copy link
Member

beikov commented Nov 21, 2023 •
edited
Loading

The implementation of checking whether an attribute path is covered by fetches is faulty in com.blazebit.persistence.view.impl.objectbuilder.ViewTypeObjectBuilder#hasSubFetches. My attempts at trying to optimize the lookup apparently cause a bug when a fetch is lexically bigger to an attribute path but has the same length.
Caused by #1465
@beikov beikov added this to the 1.6.11 milestone Nov 21, 2023
@beikov beikov self-assigned this Nov 21, 2023
beikov added a commit to beikov/blaze-persistence that referenced this issue Nov 21, 2023
@beikov
[Blazebit#1832] Fix over-fetching of entity view data with dynamic fe…
40ad816 
…tches
@beikov beikov mentioned this issue Nov 21, 2023
Merged
beikov added a commit to beikov/blaze-persistence that referenced this issue Nov 21, 2023
@beikov
[Blazebit#1832] Fix over-fetching of entity view data with dynamic fe…
fe0e317 
…tches
beikov added a commit that referenced this issue Nov 23, 2023
@beikov
[#1832] Fix over-fetching of entity view data with dynamic fetches
a05387e
beikov added a commit to beikov/blaze-persistence that referenced this issue Nov 4, 2024
@beikov
[Blazebit#1832] Fix over-fetching of entity view data with dynamic fe…
bda3446 
…tches
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@beikov beikov

Labels
component: entity-view kind: bug workaround available worth: high Implementing this has a high worth
Projects
None yet
Milestone
1.6.11
Development

Successfully merging a pull request may close this issue.

[#1832] Fix over-fetching of entity view data with dynamic fetches beikov/blaze-persistence
1 participant
@beikov