Skip to content

fix logical error when low cardinality use statistics#89343

Merged
hanfei1991 merged 2 commits intoClickHouse:masterfrom
hanfei1991:hanfei/fix-lowcard-stats
Nov 3, 2025
Merged

fix logical error when low cardinality use statistics#89343
hanfei1991 merged 2 commits intoClickHouse:masterfrom
hanfei1991:hanfei/fix-lowcard-stats

Conversation

@hanfei1991
Copy link
Copy Markdown
Member

@hanfei1991 hanfei1991 commented Oct 31, 2025

Changelog category (leave one):

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

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

Fix Statistics countmin does not support estimate data type of LowCardinality(Nullable(String)) LOGICAL_ERROR.

@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh bot commented Oct 31, 2025

Workflow [PR], commit [fa28efd]

Summary:

job_name test_name status info comment
Stateless tests (amd_msan, sequential, 2/2) failure
03141_fetches_errors_stress FAIL cidb, flaky
Integration tests (arm_binary, distributed plan, 4/4) failure
test_storage_alias_replicated/test.py::test_alias_with_replicated FAIL cidb

@clickhouse-gh clickhouse-gh bot added the pr-bugfix Pull request with bugfix, not backported by default label Oct 31, 2025
@tuanpach tuanpach self-assigned this Nov 3, 2025
@hanfei1991 hanfei1991 added this pull request to the merge queue Nov 3, 2025
Merged via the queue into ClickHouse:master with commit cafc904 Nov 3, 2025
121 of 124 checks passed
@hanfei1991 hanfei1991 deleted the hanfei/fix-lowcard-stats branch November 3, 2025 00:59
@robot-ch-test-poll robot-ch-test-poll added the pr-synced-to-cloud The PR is synced to the cloud repo label Nov 3, 2025
@PedroTadim
Copy link
Copy Markdown
Member

Please mention closes #87886 to close the issue

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 pr-synced-to-cloud The PR is synced to the cloud repo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants