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

Added query fetches search for join existence when applying JPA ordering. #2434

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

AntonMolganov
Copy link

@AntonMolganov AntonMolganov commented Feb 8, 2022

Closes #2253
See #2253

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Feb 8, 2022
@schauder
Copy link
Contributor

schauder commented Feb 9, 2022

The "See" link 404s, should that go to #2253?

@AntonMolganov
Copy link
Author

The "See" link 404s, should that go to #2253?

Fixed pull request message.
Was .../spring-data-build/... insteadof .../spring-data-jpa/...

@schauder schauder self-assigned this Feb 9, 2022
Copy link
Contributor

@schauder schauder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We would need an integration test to go with this. UserRepositoryTests would probably a good home for that.

@schauder schauder added status: waiting-for-feedback We need additional information before we can continue and removed status: waiting-for-triage An issue we've not yet triaged labels Feb 9, 2022
odrotbohm and others added 4 commits April 14, 2022 18:41
Indentation in testcases (spaces -> tabs). Additional imports.
The Auditable interface introduces Optional getters, which when combined with Query by Example results in cryptic errors. By ignoring a probe's field that contains an Optional.empty, Query by Example works properly.

NOTE: This fix actually tests outside the originally detected scope of Auditable, verifying that ALL Optional.empty() fields are properly handled.

Closes spring-projects#2176
Original pull request spring-projects#2401
Removed Given/When/Then comments.
They are of limited use, especially since it is debatable what belongs in which section.

See spring-projects#2176
Original pull request spring-projects#2401
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: waiting-for-feedback We need additional information before we can continue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Specifications with sort creates additional join even though the entity was already fetched
5 participants