Skip to content
Snippets Groups Projects
Unverified Commit 25aac414 authored by Jeff Evans's avatar Jeff Evans Committed by GitHub
Browse files

Add test for ordering when nils are present (#17914)

Add test for ordering when nils are present

Adding test to ensure that we can handle a column containing nils in an ORDER BY clause

Adding `format-name` override for `bigquery-cloud-sdk` to convert dash to underscore in test table names

Adding comment showing IDs of each row in the daily-bird-counts.edn file

Format last column as int to make Oracle happy

Add impls for `presto` and `presto-jdbc` `sorts-nil-first?` as false

Make the `sorts-nil-first?` multimethod accept a base type in order to return an answer

Vertica, at least, has a different answer for different types; see:
https://www.vertica.com/docs/9.2.x/HTML/Content/Authoring/AnalyzingData/Optimizations/NULLPlacementByAnalyticFunctions.htm#2

Updating Vertica implementation to consider the `base-type` as per its documentation

Updating all invocations of `sorts-nil-first?` in tests to pass the relevant base type
parent db7e7a0c
No related branches found
No related tags found
No related merge requests found
Showing
with 115 additions and 81 deletions
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment