Skip to content

Fix transform_null_in with sets of tuples#16722

Merged
alexey-milovidov merged 1 commit intoClickHouse:masterfrom
CurtizJ:fix-transform-null-in
Nov 6, 2020
Merged

Fix transform_null_in with sets of tuples#16722
alexey-milovidov merged 1 commit intoClickHouse:masterfrom
CurtizJ:fix-transform-null-in

Conversation

@CurtizJ
Copy link
Copy Markdown
Member

@CurtizJ CurtizJ commented Nov 5, 2020

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Changelog category (leave one):

  • Bug Fix

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Fix IN operator over several columns and tuples with enabled transform_null_in setting. Fixes #15310.

Detailed description / Documentation draft:
With enabled transform_null_in just use regular hash table, because it will be created with set variant that can handle nullable elements, and avoid null map with some strange additional flags.

@robot-clickhouse robot-clickhouse added the pr-bugfix Pull request with bugfix, not backported by default label Nov 5, 2020
@alexey-milovidov alexey-milovidov self-assigned this Nov 5, 2020
@alexey-milovidov alexey-milovidov merged commit dd83a35 into ClickHouse:master Nov 6, 2020
alexey-milovidov added a commit that referenced this pull request Nov 6, 2020
Backport #16722 to 20.11: Fix transform_null_in with sets of tuples
alexey-milovidov added a commit that referenced this pull request Nov 6, 2020
Backport #16722 to 20.10: Fix transform_null_in with sets of tuples
alexey-milovidov added a commit that referenced this pull request Nov 14, 2020
Backport #16722 to 20.8: Fix transform_null_in with sets of tuples
alexey-milovidov added a commit that referenced this pull request Nov 14, 2020
Backport #16722 to 20.9: Fix transform_null_in with sets of tuples
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-bugfix Pull request with bugfix, not backported by default

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Wrong results of query with setting transform_null_in=1.

3 participants