Skip to content

Google cloud storage support for data lakes catalogs#93866

Merged
alesapin merged 10 commits intoClickHouse:masterfrom
scanhex12:gs_storage_support
Jan 14, 2026
Merged

Google cloud storage support for data lakes catalogs#93866
alesapin merged 10 commits intoClickHouse:masterfrom
scanhex12:gs_storage_support

Conversation

@scanhex12
Copy link
Copy Markdown
Member

@scanhex12 scanhex12 commented Jan 10, 2026

Changelog category (leave one):

  • Improvement

Changelog entry (a user-readable short description of the changes that goes into CHANGELOG.md):

Google cloud storage support for data lakes

Documentation entry for user-facing changes

  • Documentation is written (mandatory for new features)

@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh bot commented Jan 10, 2026 •
edited by alesapin
Loading

Workflow [PR], commit [656e43f]

Summary: ❌

job_name test_name status info comment
Integration tests (arm_binary, distributed plan, 3/4) failure
test_storage_s3_queue/test_0.py::test_move_after_processing[another_bucket-S3Queue] FAIL cidb, issue ISSUE EXISTS
AST fuzzer (amd_debug) failure
Logical error: Unexpected return type from A. Expected B. Got C. Action: (STID: 1611-3d1e) FAIL cidb, issue ISSUE EXISTS
Integration tests (amd_asan, targeted) error IGNORED

@clickhouse-gh clickhouse-gh bot added the pr-improvement Pull request with some product improvements label Jan 10, 2026
@scanhex12 scanhex12 changed the title Google cloud storage support for data lakes Google cloud storage support for data lakes catalogs Jan 10, 2026
@scanhex12 scanhex12 added the pr-must-backport Pull request should be backported intentionally. Use this label with great care! label Jan 13, 2026
@alesapin alesapin self-assigned this Jan 13, 2026
alesapin
alesapin reviewed Jan 13, 2026
View reviewed changes
tests/integration/test_database_iceberg/rest_with_gcs/mock.py Outdated
return json.dumps({"error": {"message": message}}, ensure_ascii=False), code, "application/json; charset=utf-8"


class RequestHandler(http.server.BaseHTTPRequestHandler):
Copy link
Copy Markdown
Member

@alesapin alesapin Jan 13, 2026

Choose a reason for hiding this comment

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

Was it vibecoded?

Copy link
Copy Markdown
Member Author

@scanhex12 scanhex12 Jan 13, 2026

Choose a reason for hiding this comment

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

partially...

@scanhex12 scanhex12 requested a review from alesapin January 13, 2026 18:44
alesapin
alesapin approved these changes Jan 14, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@alesapin alesapin left a comment

Choose a reason for hiding this comment

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

Very weird failpoint but okay

@alesapin alesapin added this pull request to the merge queue Jan 14, 2026
Merged via the queue into ClickHouse:master with commit 9a5734f Jan 14, 2026
128 of 132 checks passed
This was referenced Jan 14, 2026
Closed
Closed
Closed
Closed
Closed
@robot-clickhouse robot-clickhouse added the pr-synced-to-cloud The PR is synced to the cloud repo label Jan 14, 2026
@robot-ch-test-poll1 robot-ch-test-poll1 added the pr-must-backport-synced The `*-must-backport` labels are synced into the cloud Sync PR label Jan 14, 2026
mkmkme pushed a commit to Altinity/ClickHouse that referenced this pull request Jan 16, 2026
@alesapin @mkmkme
Merge pull request ClickHouse#93866 from scanhex12/gs_storage_support
155a932 
Google cloud storage support for data lakes catalogs
@mkmkme mkmkme mentioned this pull request Jan 16, 2026
Merged
25 tasks
zvonand added a commit to Altinity/ClickHouse that referenced this pull request Jan 21, 2026
@zvonand
Merge pull request #1318 from Altinity/backports/antalya-25.8/93866
b3375be 
Antalya 25.8 Backport of ClickHouse#93866 - Google cloud storage support for data lakes catalogs
@melvynator melvynator mentioned this pull request Jan 28, 2026
Closed
@robot-ch-test-poll robot-ch-test-poll mentioned this pull request Jan 31, 2026
Closed
@robot-clickhouse-ci-1 robot-clickhouse-ci-1 added the pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore label Feb 26, 2026
@melvynator melvynator mentioned this pull request Mar 2, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alesapin alesapin alesapin approved these changes

Assignees

@alesapin alesapin

Labels

pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore pr-improvement Pull request with some product improvements pr-must-backport Pull request should be backported intentionally. Use this label with great care! pr-must-backport-synced The `*-must-backport` labels are synced into the cloud Sync PR pr-synced-to-cloud The PR is synced to the cloud repo

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@scanhex12 @alesapin @robot-clickhouse @robot-ch-test-poll1 @robot-clickhouse-ci-1