druid icon indicating copy to clipboard operation
druid copied to clipboard

SQL: Support GROUP BY and ORDER BY for NULL types.

Open gianm opened this issue 1 year ago • 4 comments

Treats SQL NULL types as strings at the native layer. This is consistent with how we treat unknown-type nulls in other contexts.

gianm avatar Apr 09 '24 18:04 gianm

@kgyrtkirk FYI I encountered an error here in decoupled planning with one of the new tests, and disabled it: https://github.com/apache/druid/pull/16252/files#diff-8d40b0c4f1c04fc1a87f672f9fb48a91568c7bd4a4eb5ceda1c9cb008a22653fR70-R76

gianm avatar Apr 09 '24 18:04 gianm

This pull request has been marked as stale due to 60 days of inactivity. It will be closed in 4 weeks if no further activity occurs. If you think that's incorrect or this pull request should instead be reviewed, please simply write any comment. Even if closed, you can still revive the PR at any time or discuss it on the [email protected] list. Thank you for your contributions.

github-actions[bot] avatar Jul 22 '24 00:07 github-actions[bot]

NOT_ENOUGH_RULES was removed in #16770; I added it back so it could be used in this PR.

gianm avatar Jul 24 '24 19:07 gianm

This pull request has been marked as stale due to 60 days of inactivity. It will be closed in 4 weeks if no further activity occurs. If you think that's incorrect or this pull request should instead be reviewed, please simply write any comment. Even if closed, you can still revive the PR at any time or discuss it on the [email protected] list. Thank you for your contributions.

github-actions[bot] avatar Sep 23 '24 00:09 github-actions[bot]

forcing PR recheck with close/open cycle

kgyrtkirk avatar Nov 28 '24 09:11 kgyrtkirk

This pull request has been marked as stale due to 60 days of inactivity. It will be closed in 4 weeks if no further activity occurs. If you think that's incorrect or this pull request should instead be reviewed, please simply write any comment. Even if closed, you can still revive the PR at any time or discuss it on the [email protected] list. Thank you for your contributions.

github-actions[bot] avatar Jan 28 '25 00:01 github-actions[bot]