Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

add(clickhouse): total storage limitation #557

Merged
merged 2 commits into from
Nov 12, 2024
Merged

add(clickhouse): total storage limitation #557

merged 2 commits into from
Nov 12, 2024

Conversation

wojcik-dorota
Copy link
Contributor

@wojcik-dorota wojcik-dorota commented Nov 11, 2024 •
edited
Loading

https://aiven.atlassian.net/browse/DDB-1361

Checklist

  • The first paragraph of the page is on one line.
  • The other lines have a line break at 90 characters.
  • I checked the output.
  • I applied the style guide.
  • My links start with /docs/.

@wojcik-dorota wojcik-dorota requested a review from a team as a code owner November 11, 2024 14:56
@wojcik-dorota wojcik-dorota self-assigned this Nov 11, 2024
@wojcik-dorota wojcik-dorota changed the title storage limitation add(clickhouse): total storage limitation Nov 11, 2024
@wojcik-dorota wojcik-dorota added the review/SME An SME must review the PR. label Nov 11, 2024
@cloudflare-workers-and-pages Cloudflare Workers and Pages
Copy link

cloudflare-workers-and-pages bot commented Nov 11, 2024 •
edited
Loading

Deploying aiven-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 9e70c13
Status: ✅  Deploy successful!
Preview URL: https://f27130b0.aiven-docs.pages.dev
Branch Preview URL: https://dorota-clickhouse-storage-pe.aiven-docs.pages.dev

View logs

@wojcik-dorota wojcik-dorota requested a review from a team November 11, 2024 15:00
@wojcik-dorota wojcik-dorota force-pushed the dorota-clickhouse-storage-per-plan-node-shard branch 2 times, most recently from bf84969 to 01cc8df Compare November 12, 2024 11:17
@wojcik-dorota wojcik-dorota force-pushed the dorota-clickhouse-storage-per-plan-node-shard branch 3 times, most recently from ecc99ba to 2a27713 Compare November 12, 2024 12:00
@wojcik-dorota wojcik-dorota force-pushed the dorota-clickhouse-storage-per-plan-node-shard branch from 2a27713 to 9e70c13 Compare November 12, 2024 12:40
@wojcik-dorota wojcik-dorota added review/TW A TW must review the PR. Comes after review/SME. and removed review/SME An SME must review the PR. labels Nov 12, 2024
harshini-rangaswamy
harshini-rangaswamy approved these changes Nov 12, 2024
View reviewed changes
Copy link
Contributor

@harshini-rangaswamy harshini-rangaswamy left a comment

Choose a reason for hiding this comment

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

Look good!

@wojcik-dorota wojcik-dorota merged commit ce1d329 into main Nov 12, 2024
4 checks passed
@wojcik-dorota wojcik-dorota deleted the dorota-clickhouse-storage-per-plan-node-shard branch November 12, 2024 15:09
wojcik-dorota added a commit that referenced this pull request Nov 19, 2024
@wojcik-dorota
add(clickhouse): total storage limitation (#557)
62e6456
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@harshini-rangaswamy harshini-rangaswamy harshini-rangaswamy approved these changes

@Khatskevich Khatskevich Khatskevich approved these changes

Assignees

@wojcik-dorota wojcik-dorota

Labels
review/TW A TW must review the PR. Comes after review/SME. service/clickhouse
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@wojcik-dorota @Khatskevich @harshini-rangaswamy