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

Pageable sorting by join property excludes null matches with Specifications [DATAJPA-277] #690

Closed
spring-projects-issues opened this issue Dec 17, 2012 · 3 comments
Assignees
Milestone

Comments

@spring-projects-issues
Copy link

@spring-projects-issues spring-projects-issues commented Dec 17, 2012

P. J. Reed opened DATAJPA-277 and commented

If you have a repository that extends JpaSpecficationExecutor and use the findAll method with a Specification, the resulting query excludes null matches from the result. This may be related to issue DATAJPA-252.

I've attached a test case that demonstrates how DATAJPA-252 is fixed but this is still an issue


Affects: 1.2 GA

Attachments:

Issue Links:

  • DATAJPA-346 Sort by property of an associated object doesn't work. Join type should be LEFT OUTER JOIN for sorting on associated objects

Referenced from: commits bc6e2a7, 52cccbb

Backported to: 1.2.1

1 votes, 5 watchers

@spring-projects-issues
Copy link
Author

@spring-projects-issues spring-projects-issues commented Jan 4, 2013

Deepak Kaul commented

Are there any plans to fix this in near term?

@spring-projects-issues
Copy link
Author

@spring-projects-issues spring-projects-issues commented Jan 17, 2013

Oliver Drotbohm commented

We now also use left joins in case the path referenced in the sorting criteria refers to an entity. Snapshots should be available shortly

@spring-projects-issues
Copy link
Author

@spring-projects-issues spring-projects-issues commented Dec 12, 2018

Daniel Costa commented

Oliver Drotbohm This is still happening to me on springdata-jpa 1.11.0. On repository that extends JpaSpecificationExecutor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants