Skip to content

Verify that interface-based projections work with subselects #420

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
Closed
wants to merge 2 commits into from
Closed

Verify that interface-based projections work with subselects #420

wants to merge 2 commits into from

Conversation

schauder
Copy link
Contributor

This pr adds a failing test, reproducing the issue.

Issue: https://jira.spring.io/browse/DATAJPA-1713

@spring-projects-issues spring-projects-issues mentioned this pull request Dec 30, 2020
Closed
@gregturn gregturn changed the base branch from master to main April 15, 2021 18:45
@gregturn gregturn changed the title *** DO NOT MERGE *** DATAJPA-1713 - Projections don't work with subselect. Verify that interface-based projections work with subselects May 31, 2023
@gregturn gregturn linked an issue May 31, 2023 that may be closed by this pull request
Projection fails when using subquery [DATAJPA-1713] #2008
Closed
gregturn added a commit that referenced this pull request May 31, 2023
@gregturn
Verify that projections work with subqueries.
7b5a57d 
Interface-based projections used to NOT work when the query had a subquery in it. But with the new query parser, it already handles it. So this simply captures the test providing that corner case.

See #2008
Original Pull Request: #420
gregturn added a commit that referenced this pull request May 31, 2023
@gregturn
Verify that projections work with subqueries.
9b50425 
Interface-based projections used to NOT work when the query had a subquery in it. But with the new query parser, it already handles it. So this simply captures the test providing that corner case.

See #2008
Original Pull Request: #420
@gregturn gregturn closed this May 31, 2023
@gregturn gregturn self-assigned this May 31, 2023
gregturn added a commit that referenced this pull request Jun 1, 2023
@gregturn
Verify that projections work with subqueries.
28de271 
Interface-based projections used to NOT work when the query had a subquery in it. But with the new query parser, it already handles it. So this simply captures the test providing that corner case.

See #2008
Original Pull Request: #420
gregturn added a commit that referenced this pull request Jun 1, 2023
@gregturn
Verify that projections work with subqueries.
280726f 
Interface-based projections used to NOT work when the query had a subquery in it. But with the new query parser, it already handles it. So this simply captures the test providing that corner case.

See #2008
Original Pull Request: #420
@gregturn gregturn deleted the issue/DATAJPA-1713 branch June 15, 2023 18:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@gregturn gregturn

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Projection fails when using subquery [DATAJPA-1713]
2 participants
@schauder @gregturn