Skip to content

increase default max_suspicious_broken_parts to 100#41619

Merged
KochetovNicolai merged 1 commit intoClickHouse:masterfrom
den-crane:patch-50
Sep 22, 2022
Merged

increase default max_suspicious_broken_parts to 100#41619
KochetovNicolai merged 1 commit intoClickHouse:masterfrom
den-crane:patch-50

Conversation

@den-crane
Copy link
Contributor

@den-crane den-crane commented Sep 21, 2022

closes: #41423

We often see complaints about startup issue Suspiciously many (12 parts, 0.00 B in total) broken parts to remove while maximum allowed broken parts count is 10 and in 100% cases it's because of an unexpected restart and files with 0 size or parts without half of files. The maximum number that I saw in public chats or from Altinity clients is 70 broken parts.

(it seems the modern hardware (ssd) + arbitrary partitioning allow to create a more parts per second than it was in 2016 when max_suspicious_broken_parts = 10 was good enough)

Personally I set max_suspicious_broken_parts = 100.

Anyway this behaviour is secured by max_suspicious_broken_parts_bytes │ 1073741824

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

@robot-clickhouse robot-clickhouse added the pr-not-for-changelog This PR should not be mentioned in the changelog label Sep 21, 2022
@KochetovNicolai KochetovNicolai merged commit 3b63719 into ClickHouse:master Sep 22, 2022
@KochetovNicolai KochetovNicolai self-assigned this Sep 22, 2022
@ramazanpolat
Copy link
Contributor

@den-crane This setting does not exist in system.settings table. max_suspicious_broken_parts_bytes also missing from settings table. Any idea why?

@den-crane
Copy link
Contributor Author

@ramazanpolat
It's a merge_tree setting.

select * from system.merge_tree_settings  where name like '%max_suspicious_broken_parts%';

┌─name──────────────────────────────┬─value──────┬─changed─┬─description──────────────────────────────────────────────────────┬─type───┐
│ max_suspicious_broken_parts       │ 1000 │ Max broken parts, if more - deny automatic deletion.             │ UInt64 │
│ max_suspicious_broken_parts_bytes │ 10737418240 │ Max size of all broken parts, if more - deny automatic deletion. │ UInt64 │
└───────────────────────────────────┴────────────┴─────────┴──────────────────────────────────────────────────────────────────┴────────┘

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.

max_suspicious_broken_parts = 10 (default) is too low

4 participants