Skip to content

Fix 43622#43731

Merged
evillique merged 1 commit intoClickHouse:masterfrom
amosbird:issue_43622
Nov 29, 2022
Merged

Fix 43622#43731
evillique merged 1 commit intoClickHouse:masterfrom
amosbird:issue_43622

Conversation

@amosbird
Copy link
Copy Markdown
Collaborator

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

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

Fix extremely rare case when user use LowCardinality(float) as primary key and it contains nan. This fixes #43622 . There is no need for changelog.

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

@amosbird amosbird marked this pull request as ready for review November 27, 2022 10:42
@robot-clickhouse-ci-1 robot-clickhouse-ci-1 added the pr-not-for-changelog This PR should not be mentioned in the changelog label Nov 27, 2022
@evillique evillique self-assigned this Nov 27, 2022
@evillique
Copy link
Copy Markdown
Member

AST fuzzer (msan): #43736
Other failures are not related.

@evillique evillique merged commit e6e6afe into ClickHouse:master Nov 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-not-for-changelog This PR should not be mentioned in the changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Logical error: Invalid Field get from type Float64 to type Int64 in KeyCondition

3 participants