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

Multiple roots generated for limit mapping with intermediate association #1811

Open
beikov opened this issue Oct 5, 2023 · 2 comments · May be fixed by #1812
Open

Multiple roots generated for limit mapping with intermediate association #1811

beikov opened this issue Oct 5, 2023 · 2 comments · May be fixed by #1812
Assignees
@beikov
Labels
component: core kind: bug worth: high Implementing this has a high worth
Milestone
2.0.1

Comments

@beikov
Copy link
Member

beikov commented Oct 5, 2023

In an entity view mapping like e.g.

@Mapping("page.pageRevisions")
@Limit(limit = "1", order = ["id DESC"])
PageRevisionGraphQlView getLastRevision();

we seem to generate an additional root join node, so paginating fails.
@beikov beikov added kind: bug component: core worth: high Implementing this has a high worth labels Oct 5, 2023
@beikov beikov added this to the 1.6.11 milestone Oct 5, 2023
beikov added a commit to beikov/blaze-persistence that referenced this issue Oct 5, 2023
@beikov
[Blazebit#1811] Don't generate root for intermediate from node of cor…
64fa0a9 
…related join path
@beikov beikov linked a pull request Oct 5, 2023 that will close this issue
Open
@beikov beikov self-assigned this Oct 5, 2023
beikov added a commit to beikov/blaze-persistence that referenced this issue Oct 5, 2023
@beikov
[Blazebit#1811] Don't generate root for intermediate from node of cor…
cf1caf3 
…related join path
@EugenMayer
Copy link
Contributor

@beikov should i have somewhat tested this? Not sure which cause we have been working on here

@beikov
Copy link
Member Author

beikov commented Dec 11, 2023

Don't worry about this. I think I have a test for this, but did not find the time yet to fix this as it is rather complex.

@beikov beikov modified the milestones: 1.6.11, 1.6.12 Jan 10, 2024
beikov added a commit to beikov/blaze-persistence that referenced this issue Jul 15, 2024
@beikov
[Blazebit#1811] Don't generate root for intermediate from node of cor…
aaf9962 
…related join path
@beikov beikov modified the milestones: 1.6.12, 1.6.13 Aug 2, 2024
@beikov beikov modified the milestones: 1.6.13, 2.0.1 Nov 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@beikov beikov

Labels
component: core kind: bug worth: high Implementing this has a high worth
Projects
None yet
Milestone
2.0.1
2 participants
@EugenMayer @beikov