Skip to content

Better exception about reaching the memory threshold during inserts#87570

Merged
yariks5s merged 8 commits intoClickHouse:masterfrom
ylw510:better_output_format
Oct 9, 2025
Merged

Better exception about reaching the memory threshold during inserts#87570
yariks5s merged 8 commits intoClickHouse:masterfrom
ylw510:better_output_format

Conversation

@ylw510
Copy link
Copy Markdown
Contributor

@ylw510 ylw510 commented Sep 24, 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 into CHANGELOG.md):

Documentation entry for user-facing changes

  • Documentation is written (mandatory for new features)

@yariks5s yariks5s self-assigned this Sep 24, 2025
@yariks5s yariks5s added the can be tested Allows running workflows for external contributors label Sep 24, 2025
@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh Bot commented Sep 24, 2025

Workflow [PR], commit [723c609]

Summary:

@clickhouse-gh clickhouse-gh Bot added the pr-not-for-changelog This PR should not be mentioned in the changelog label Sep 24, 2025
@alexey-milovidov
Copy link
Copy Markdown
Member

@yariks5s, change the PR title and add the information where the threshold can be configured.
Edit the message to sound better, e.g.: The amount of free space ({}) on disk {} is less than the configured threshold ({}). The threshold can be configured by ... The total disk capacity is {}.

@ylw510
Copy link
Copy Markdown
Contributor Author

ylw510 commented Sep 24, 2025

@yariks5s, change the PR title and add the information where the threshold can be configured. Edit the message to sound better, e.g.: The amount of free space ({}) on disk {} is less than the configured threshold ({}). The threshold can be configured by ... The total disk capacity is {}.

done!

@yariks5s yariks5s changed the title Better output format Better exception about reaching the memory threshold during inserts Oct 6, 2025
@yariks5s yariks5s added this pull request to the merge queue Oct 9, 2025
Merged via the queue into ClickHouse:master with commit 4451e1d Oct 9, 2025
123 checks passed
@robot-clickhouse-ci-2 robot-clickhouse-ci-2 added the pr-synced-to-cloud The PR is synced to the cloud repo label Oct 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

can be tested Allows running workflows for external contributors 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.

4 participants