Skip to content

Add test for #78907#80840

Merged
vdimir merged 5 commits intomasterfrom
vdimir/fix_78907
Jun 18, 2025
Merged

Add test for #78907#80840
vdimir merged 5 commits intomasterfrom
vdimir/fix_78907

Conversation

@vdimir
Copy link
Member

@vdimir vdimir commented May 26, 2025

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):

...

close #78907

@clickhouse-gh
Copy link
Contributor

clickhouse-gh bot commented May 26, 2025

Workflow [PR], commit [16fcb3e]

@clickhouse-gh clickhouse-gh bot added the pr-not-for-changelog This PR should not be mentioned in the changelog label May 26, 2025
@hanfei1991 hanfei1991 self-assigned this May 27, 2025
@vdimir vdimir enabled auto-merge June 2, 2025 16:24
@clickhouse-gh
Copy link
Contributor

clickhouse-gh bot commented Jun 17, 2025

Workflow [PR], commit [ae465c5]

Summary:

@novikd
Copy link
Member

novikd commented Jun 17, 2025

For some reason, the test query failed with OOM with parallel replicas. It seems ClickHouse allocated a lot of memory for the hash table.

Query memory limit exceeded: would use 28.91 MiB (attempt to allocate chunk of 1.00 MiB bytes), maximum: 28.61 MiB. (MEMORY_LIMIT_EXCEEDED) (version 25.6.1.1) (from [::1]:51784) (comment: 03522_join_using_bug_78907.sql)

I removed max_memory_usage = 30000000 to fix the test.

@vdimir vdimir added this pull request to the merge queue Jun 18, 2025
Merged via the queue into master with commit 65512d4 Jun 18, 2025
236 of 240 checks passed
@vdimir vdimir deleted the vdimir/fix_78907 branch June 18, 2025 10:24
@robot-ch-test-poll2 robot-ch-test-poll2 added the pr-synced-to-cloud The PR is synced to the cloud repo label Jun 18, 2025
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 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.

Bad cast with analyzer_compatibility_join_using_top_level_identifier

4 participants