Skip to content
This repository has been archived by the owner on Jan 29, 2024. It is now read-only.

Clickhouse max_table_to_drop #2215

Merged

Conversation

Khatskevich
Copy link
Contributor

  1. max_table_size_to_drop and max_partition_size_to_drop are changed. The new value differs from the default, so, it should be mentioned. this commit adds the docs
  2. fix broken links

@Khatskevich Khatskevich marked this pull request as ready for review November 8, 2023 23:47
@Khatskevich Khatskevich force-pushed the khatskevich-clickhouse-max-table-to-drop branch from 3330e47 to aa9f982 Compare November 8, 2023 23:56
@wojcik-dorota wojcik-dorota self-requested a review November 9, 2023 07:25
@Khatskevich Khatskevich force-pushed the khatskevich-clickhouse-max-table-to-drop branch from a4ca9df to 6ed5f52 Compare November 9, 2023 18:46
Copy link
Contributor

@wojcik-dorota wojcik-dorota left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Many thanks for the update :-) Please see my comments.

@Khatskevich Khatskevich force-pushed the khatskevich-clickhouse-max-table-to-drop branch from 6ed5f52 to 69a2f7c Compare November 17, 2023 01:58
Copy link
Contributor

@wojcik-dorota wojcik-dorota left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Appreciate you effort 🙌🏻

@wojcik-dorota wojcik-dorota merged commit e5d26d2 into aiven:main Nov 17, 2023
3 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants