Skip to content

Document Delta Lake support for GCS#12382

Merged
ebyhr merged 1 commit intotrinodb:masterfrom
alexjo2144:delta/gcs-read-docs
May 20, 2022
Merged

Document Delta Lake support for GCS#12382
ebyhr merged 1 commit intotrinodb:masterfrom
alexjo2144:delta/gcs-read-docs

Conversation

@alexjo2144
Copy link
Copy Markdown
Member

Description

Documentation only

Related issues, pull requests, and links

Read supported added by #12144

Documentation

( ) No documentation is needed.
(x) Sufficient documentation is included in this PR.
( ) Documentation PR is available with #prnumber.
( ) Documentation issue #issuenumber is filed, and can be handled later.

Release notes

(x) No release notes entries required.
( ) Release notes entries required with the following suggested text:

@findepi
Copy link
Copy Markdown
Member

findepi commented May 16, 2022

LGTM.
@mosabua PTAL

@alexjo2144
Copy link
Copy Markdown
Member Author

Updated to include CTAS and CREATE TABLE support.

@findepi findepi requested review from mosabua and removed request for mosabua May 17, 2022 14:42
Copy link
Copy Markdown
Contributor

@grantatspothero grantatspothero left a comment

Choose a reason for hiding this comment

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

👍

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

ultimately we should consider moving all the GCS stuff into a separate page .. but that can be a separate PR

@alexjo2144 alexjo2144 force-pushed the delta/gcs-read-docs branch from d9741bf to 40db90e Compare May 17, 2022 21:03
@alexjo2144 alexjo2144 requested a review from mosabua May 17, 2022 21:03
@ebyhr ebyhr merged commit ac079a4 into trinodb:master May 20, 2022
@github-actions github-actions bot added this to the 382 milestone May 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

5 participants