search

cap-js / cds-dbs

Monorepo for SQL Database Services for CAP
https://cap.cloud.sap/docs/
Apache License 2.0
37 stars 11 forks source link

fix: enable nulls first #893

Closed David-Kunz closed 1 week ago

David-Kunz commented 1 week ago

OData demands NULLS FIRST for ASC and NULLS LAST for DESC. SQLite, HANA behave like that, for Postgres the default is the opposite which is now changed.

David-Kunz commented 1 week ago

could you add a test for this in our compliance test suite (then it would be tested for all dbs) or at whatever place you may think is appropriate? Thanks :)

of course, I just forgot to put this PR as a draft PR