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

update(clickhouse): integration data flow and residency #567

Merged
merged 3 commits into from
Nov 15, 2024
Merged

update(clickhouse): integration data flow and residency #567

merged 3 commits into from
Nov 15, 2024

Conversation

wojcik-dorota
Copy link
Contributor

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

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

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 15, 2024 13:13
@wojcik-dorota wojcik-dorota changed the title Dorota clickhouse integration data flow and residency update(clickhouse): integration data flow and residency Nov 15, 2024
@wojcik-dorota wojcik-dorota self-assigned this Nov 15, 2024
@wojcik-dorota wojcik-dorota added the review/SME An SME must review the PR. label Nov 15, 2024
@cloudflare-workers-and-pages Cloudflare Workers and Pages
Copy link

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

Deploying aiven-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: c6b6374
Status: ✅  Deploy successful!
Preview URL: https://38cc9c45.aiven-docs.pages.dev
Branch Preview URL: https://dorota-clickhouse-integratio.aiven-docs.pages.dev

View logs

aris-aiven
aris-aiven approved these changes Nov 15, 2024
View reviewed changes
Copy link
Contributor

@aris-aiven aris-aiven left a comment

Choose a reason for hiding this comment

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

👍

@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 15, 2024
harshini-rangaswamy
harshini-rangaswamy approved these changes Nov 15, 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.

Looks good to me! There is one minor suggestion, but you know this service is better.

@wojcik-dorota wojcik-dorota merged commit a3144d7 into main Nov 15, 2024
4 checks passed
@wojcik-dorota wojcik-dorota deleted the dorota-clickhouse-integration-data-flow-and-residency branch November 15, 2024 14:29
wojcik-dorota added a commit that referenced this pull request Nov 19, 2024
@wojcik-dorota
update(clickhouse): integration data flow and residency (#567)
9e34dbe
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aris-aiven aris-aiven aris-aiven approved these changes

@harshini-rangaswamy harshini-rangaswamy harshini-rangaswamy 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 @harshini-rangaswamy @aris-aiven