Skip to content

Fix array join on replicated map column#89717

Merged
Avogar merged 4 commits intoClickHouse:masterfrom
Avogar:fix-array-join-on-map
Nov 11, 2025
Merged

Fix array join on replicated map column#89717
Avogar merged 4 commits intoClickHouse:masterfrom
Avogar:fix-array-join-on-map

Conversation

@Avogar
Copy link
Copy Markdown
Member

@Avogar Avogar commented Nov 7, 2025 •
edited by clickhouse-gh bot
Loading

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 possible logical error during ARRAY JOIN on Map column with enabled enable_lazy_columns_replication. Closes #89705

Details

Resolves #89705

@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh bot commented Nov 7, 2025 •
edited
Loading

Workflow [PR], commit [27abd9a]

Summary: ❌

job_name test_name status info comment
Stateless tests (arm_binary, parallel) failure
00076_system_columns_bytes FAIL cidb
AST fuzzer (amd_ubsan) failure
Logical error: 'function_node_type->equals(*removeLowCardinality(node->getResultType()))'. FAIL cidb

@clickhouse-gh clickhouse-gh bot added the pr-bugfix Pull request with bugfix, not backported by default label Nov 7, 2025
@alexey-milovidov alexey-milovidov changed the title Fix arry join on replicated map column Fix array join on replicated map column Nov 7, 2025
@Avogar
Copy link
Copy Markdown
Member Author

Avogar commented Nov 11, 2025 •
edited
Loading

Stateless tests (arm_binary, parallel) - #85363
AST fuzzer (amd_ubsan) - #89851

@Avogar Avogar added this pull request to the merge queue Nov 11, 2025
Merged via the queue into ClickHouse:master with commit 97c881a Nov 11, 2025
128 of 131 checks passed
@Avogar Avogar deleted the fix-array-join-on-map branch November 11, 2025 13:40
@robot-ch-test-poll1 robot-ch-test-poll1 added the pr-synced-to-cloud The PR is synced to the cloud repo label Nov 11, 2025
@robot-ch-test-poll3 robot-ch-test-poll3 added the pr-must-backport-synced The `*-must-backport` labels are synced into the cloud Sync PR label Nov 11, 2025
@robot-ch-test-poll3 robot-ch-test-poll3 mentioned this pull request Nov 11, 2025
Merged
robot-clickhouse-ci-2 added a commit that referenced this pull request Nov 11, 2025
@robot-clickhouse-ci-2
Merge pull request #89862 from ClickHouse/cherrypick/25.10/89717
924cc9a 
Cherry pick #89717 to 25.10: Fix array join on replicated map column
robot-clickhouse added a commit that referenced this pull request Nov 11, 2025
@robot-clickhouse
Backport #89717 to 25.10: Fix array join on replicated map column
0b0588d
@robot-clickhouse-ci-2 robot-clickhouse-ci-2 mentioned this pull request Nov 11, 2025
Merged
@robot-ch-test-poll4 robot-ch-test-poll4 added the pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore label Nov 11, 2025
clickhouse-gh bot added a commit that referenced this pull request Nov 11, 2025
@clickhouse-gh
Merge pull request #89871 from ClickHouse/backport/25.10/89717
a6facdf 
Backport #89717 to 25.10: Fix array join on replicated map column
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alexey-milovidov alexey-milovidov alexey-milovidov approved these changes

Assignees

@alexey-milovidov alexey-milovidov

Labels

pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore pr-bugfix Pull request with bugfix, not backported by default pr-must-backport-synced The `*-must-backport` labels are synced into the cloud Sync PR pr-synced-to-cloud The PR is synced to the cloud repo v25.10-must-backport

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Bad cast from type DB::ColumnReplicated to DB::ColumnMap

5 participants

@Avogar @alexey-milovidov @robot-ch-test-poll1 @robot-ch-test-poll4 @robot-ch-test-poll3