Skip to content

Support for Sort.NullHandling [DATAJDBC-599] #821

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
spring-projects-issues opened this issue Sep 11, 2020 · 1 comment
Closed

Support for Sort.NullHandling [DATAJDBC-599] #821

spring-projects-issues opened this issue Sep 11, 2020 · 1 comment
Assignees
@ctailor2
Labels
status: ideal-for-contribution An issue that a contributor can help us with type: enhancement A general enhancement

Comments

@spring-projects-issues
Copy link

Veli-Pekka Muli opened DATAJDBC-599 and commented

No further details from DATAJDBC-599
@spring-projects-issues
Copy link
Author

Mark Paluch commented

We can add support for Oracle, Postgres, H2 and MySQL/MariaDB (to some extent) to consider NullHandling. SQL Server (and a few other databases) don't support specifying NullHandling in the ORDER BY clause. We should reject these specifications when running such a query

@spring-projects-issues spring-projects-issues added status: ideal-for-contribution An issue that a contributor can help us with type: enhancement A general enhancement labels Dec 31, 2020
@ctailor2 ctailor2 self-assigned this Feb 3, 2022
@ctailor2 ctailor2 mentioned this issue Feb 3, 2022
Closed
schauder added a commit that referenced this issue Feb 8, 2022
@schauder
Polishing.
4989872 
Rename "null handling" to "null precedence".
This is somewhat inconsistent with commons null handling, but more descriptive.

Minor formatting.

Original pull request #1156
See #821
schauder pushed a commit that referenced this issue Feb 15, 2022
@ctailor2 @schauder
Null precedence is now supported if the underlying database supports it.
60a59b6 
Original pull request #1156
Closes #821
schauder added a commit that referenced this issue Feb 15, 2022
@schauder
Polishing.
4066df9 
Rename "null handling" to "null precedence".
This is somewhat inconsistent with commons null handling, but more descriptive.

Minor formatting.

Original pull request #1156
See #821
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ctailor2 ctailor2

Labels
status: ideal-for-contribution An issue that a contributor can help us with type: enhancement A general enhancement
Projects
None yet
Milestone
No milestone
2 participants
@ctailor2 @spring-projects-issues