Skip to content

make_list_with_nulls_bug1009 fix#80

Merged
SadiHassan merged 1 commit intoKusto-phase2from
Kusto-phase2-make_list_bug_1009
Sep 20, 2022
Merged

make_list_with_nulls_bug1009 fix#80
SadiHassan merged 1 commit intoKusto-phase2from
Kusto-phase2-make_list_bug_1009

Conversation

@SadiHassan
Copy link
Copy Markdown

Changelog category (leave one):

  • Bug Fix (user-visible misbehavior in official stable or prestable release)

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):

...

Bug fix for make_list_with_nulls:
input query: Customers | summarize t = make_list_with_nulls(Age) by FirstName
output:
┌─FirstName─┬─t─────────────────────────┐

│ bbb │ [20,10,NULL,NULL] │
│ ccc │ [3,1,2,NULL,NULL,NULL] │
│ aaa │ [200,100,200,300,300,100] │
└───────────┴───────────────────────────┘

Note: please use non-string columns because KQL doesn't support NULL for string datatype.
Also, the order of NULL is not defined because If the input to the summarize operator is not sorted, the order of elements in the resulting array is undefined.

Information about CI checks: https://clickhouse.com/docs/en/development/continuous-integration/

@SadiHassan SadiHassan requested a review from kashwy September 19, 2022 18:22
@SadiHassan SadiHassan merged commit 568bdc5 into Kusto-phase2 Sep 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants