Skip to content

support for https connector for clickhouse#8515

Merged
NamanMahor merged 2 commits intomainfrom
support_for_https_connector_for_clickhouse
Dec 17, 2025
Merged

support for https connector for clickhouse#8515
NamanMahor merged 2 commits intomainfrom
support_for_https_connector_for_clickhouse

Conversation

@NamanMahor
Copy link
Contributor

@NamanMahor NamanMahor commented Dec 16, 2025

support for https connector for clickhouse
https://rilldata.slack.com/archives/C07710F6932/p1765469954095099

Checklist:

  • Covered by tests
  • Ran it and it works as intended
  • Reviewed the diff before requesting a review
  • Checked for unhandled edge cases
  • Linked the issues it closes
  • Checked if the docs need to be updated. If so, create a separate Linear DOCS issue
  • Intend to cherry-pick into the release branch
  • I'm proud of this work!

Copy link
Contributor

@begelundmuller begelundmuller left a comment

Choose a reason for hiding this comment

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

LGTM – can you also add an integration test for it?

@NamanMahor NamanMahor merged commit 9f902f7 into main Dec 17, 2025
13 of 14 checks passed
@NamanMahor NamanMahor deleted the support_for_https_connector_for_clickhouse branch December 17, 2025 10:51
@NamanMahor NamanMahor self-assigned this Jan 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants