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): azureBlobStorage table engine and function #610

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

add(clickhouse): azureBlobStorage table engine and function #610

wants to merge 1 commit into from
+11 −7

Conversation

wojcik-dorota
Copy link
Contributor

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

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

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 15:07
@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: 5d2e34d
Status: ✅  Deploy successful!
Preview URL: https://34b43190.aiven-docs.pages.dev
Branch Preview URL: https://dorota-clickhouse-azure-blob.aiven-docs.pages.dev

View logs

@wojcik-dorota wojcik-dorota self-assigned this Dec 18, 2024
@wojcik-dorota wojcik-dorota added review/SME An SME must review the PR. service/clickhouse review/awaitingRelease All reviews are completed. and removed service/clickhouse labels Dec 18, 2024
@wojcik-dorota wojcik-dorota force-pushed the dorota-clickhouse-azure-blob-storage-table-engine-function branch from 26e2e03 to 5d2e34d Compare December 18, 2024 15:29
@tilman-aiven
Copy link

Lets not merge it yet, the last related PR with the changes

@wojcik-dorota
Copy link
Contributor Author

wojcik-dorota commented Dec 19, 2024 •
edited by jira bot
Loading

Lets not merge it yet, the last related PR with the changes

Won't merge it until you approve :-)

tilman-aiven
tilman-aiven approved these changes Jan 8, 2025
View reviewed changes
Copy link

@tilman-aiven tilman-aiven left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tilman-aiven tilman-aiven tilman-aiven approved these changes

@ArthurFlag ArthurFlag ArthurFlag approved these changes

Assignees

@wojcik-dorota wojcik-dorota

Labels
review/awaitingRelease All reviews are completed. review/SME An SME must review the PR. service/clickhouse
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@wojcik-dorota @tilman-aiven @ArthurFlag