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): local on-disk cache for remote files #609

Merged
merged 3 commits into from
Jan 8, 2025
Merged

add(clickhouse): local on-disk cache for remote files #609

merged 3 commits into from
Jan 8, 2025
+76 −1

Conversation

wojcik-dorota
Copy link
Contributor

@wojcik-dorota wojcik-dorota commented Dec 18, 2024 •
edited
Loading

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

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 December 18, 2024 10:40
@wojcik-dorota wojcik-dorota self-assigned this Dec 18, 2024
@wojcik-dorota wojcik-dorota added the review/SME An SME must review the PR. label Dec 18, 2024
@cloudflare-workers-and-pages Cloudflare Workers and Pages
Copy link

cloudflare-workers-and-pages bot commented Dec 18, 2024 •
edited
Loading

Deploying aiven-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 5d6caa2
Status: ✅  Deploy successful!
Preview URL: https://286ac6b8.aiven-docs.pages.dev
Branch Preview URL: https://dorota-clickhouse-local-on-d.aiven-docs.pages.dev

View logs

@wojcik-dorota wojcik-dorota force-pushed the dorota-clickhouse-local-on-disk-cache-tiered-storage branch from eeb6d3b to 5c45a5c Compare December 18, 2024 10:50
@wojcik-dorota wojcik-dorota requested a review from a team December 18, 2024 12:48
@wojcik-dorota wojcik-dorota force-pushed the dorota-clickhouse-local-on-disk-cache-tiered-storage branch from 5c45a5c to e79f126 Compare December 18, 2024 13:31
joelynch

This comment was marked as resolved.

Sign in to view

@joelynch
Copy link
Contributor

Needs approval from a code owner.

@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 Dec 19, 2024
@wojcik-dorota wojcik-dorota added review/awaitingRelease All reviews are completed. and removed review/TW A TW must review the PR. Comes after review/SME. labels Dec 19, 2024
@joelynch
Copy link
Contributor

joelynch commented Jan 8, 2025

This has been released right @sjamgade ? Can we merge this?

@sjamgade Swarmia
Copy link

sjamgade commented Jan 8, 2025

@joelynch yes

@wojcik-dorota wojcik-dorota merged commit 09cdc56 into main Jan 8, 2025
4 checks passed
@wojcik-dorota wojcik-dorota deleted the dorota-clickhouse-local-on-disk-cache-tiered-storage branch January 8, 2025 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ArthurFlag ArthurFlag ArthurFlag approved these changes

@joelynch joelynch joelynch approved these changes

@sjamgade sjamgade Awaiting requested review from sjamgade

Assignees

@wojcik-dorota wojcik-dorota

Labels
review/awaitingRelease All reviews are completed. service/clickhouse
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@wojcik-dorota @joelynch @sjamgade @ArthurFlag