Skip to content

Fix null check for querydsl predicate executor#4275

Open
stupid58fly wants to merge 1 commit into
spring-projects:mainfrom
stupid58fly:bugfix/fix-null-check-for-querydsl-predicate-excecutor
Open

Fix null check for querydsl predicate executor#4275
stupid58fly wants to merge 1 commit into
spring-projects:mainfrom
stupid58fly:bugfix/fix-null-check-for-querydsl-predicate-excecutor

Conversation

@stupid58fly

Copy link
Copy Markdown

This PR fixes the issue for querydsl scroll where one of keyset position values is null.
Before fix querydsl generates query with column = null and after fix it will be column is null.

  • You have read the Spring Data contribution guidelines.
  • You use the code formatters provided here and have them applied to your changes. Don’t submit any formatting related changes.
  • You submit test cases (unit or integration tests) that back your changes.
  • You added yourself as author in the headers of the classes you touched. Amend the date range in the Apache license header if needed. For new types, add the license header (copy from another file and set the current year only).

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jun 5, 2026
Signed-off-by: Ilya Bakaev <stupid58fly@gmail.com>
@stupid58fly stupid58fly force-pushed the bugfix/fix-null-check-for-querydsl-predicate-excecutor branch from c1d6378 to 51d8211 Compare June 5, 2026 16:59
@mp911de

mp911de commented Jun 8, 2026

Copy link
Copy Markdown
Member

Can we have simpler tests, ideally that demonstrate the problem and test the query itself rather than the effect?

@stupid58fly

Copy link
Copy Markdown
Author

Can we have simpler tests, ideally that demonstrate the problem and test the query itself rather than the effect?

I tried to write this test, but didn't succeed.
There is no such tests for QueryDSL, at least I could not find it.
I have tried to do that, but couldn't find any point to join the query rendering.
Not sure it is possible to implement such test without refactoring QueryDSL implementation.
If you have any suggestions, could you please share?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: waiting-for-triage An issue we've not yet triaged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants