Skip to content

externalconn: add gs support to External Connections#85964

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
adityamaru:gcs-ec-support
Aug 16, 2022
Merged

externalconn: add gs support to External Connections#85964
craig[bot] merged 1 commit intocockroachdb:masterfrom
adityamaru:gcs-ec-support

Conversation

@adityamaru
Copy link
Copy Markdown
Contributor

@adityamaru adityamaru commented Aug 11, 2022

This change registers Google Storage gs as a supported
External Connection.

Release note (sql change): Users can now
CREATE EXTERNAL CONNECTION to represent an underlying
google storage resource.

Release justification: low risk change to new functionality around External Connections

@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@adityamaru adityamaru force-pushed the gcs-ec-support branch 4 times, most recently from 7c5d29f to db6877a Compare August 11, 2022 18:34
@adityamaru adityamaru marked this pull request as ready for review August 11, 2022 18:50
@adityamaru adityamaru requested review from a team as code owners August 11, 2022 18:50
@adityamaru adityamaru requested a review from a team August 11, 2022 18:50
@adityamaru adityamaru requested review from a team as code owners August 11, 2022 18:50
@adityamaru adityamaru requested review from a team, ajwerner, benbardin, miretskiy, msbutler and rhu713 and removed request for a team, ajwerner and msbutler August 11, 2022 18:50
@adityamaru
Copy link
Copy Markdown
Contributor Author

Commit 3 is the only new one.

@adityamaru
Copy link
Copy Markdown
Contributor Author

@rhu713 @benbardin this should be easier to review now. I'll run our nightly unit test job to ensure it succeeds before merging.

@adityamaru adityamaru force-pushed the gcs-ec-support branch 2 times, most recently from b1678b1 to 33550ce Compare August 15, 2022 20:11
This change registers Google Storage `gs` as a supported
External Connection.

Release note (sql change): Users can now
`CREATE EXTERNAL CONNECTION` to represent an underlying
google storage resource.
@adityamaru
Copy link
Copy Markdown
Contributor Author

@adityamaru
Copy link
Copy Markdown
Contributor Author

TFTR!

bors r=rhu713

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Aug 16, 2022

This PR was included in a batch that successfully built, but then failed to merge into master. It will not be retried.

Additional information:

{"message":"1 review requesting changes and 1 approving review by reviewers with write access.","documentation_url":"https://docs.github.com/articles/about-protected-branches"}

@adityamaru
Copy link
Copy Markdown
Contributor Author

bors r=rhu713

@craig craig bot merged commit 7c38417 into cockroachdb:master Aug 16, 2022
@craig
Copy link
Copy Markdown
Contributor

craig bot commented Aug 16, 2022

Build succeeded:

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.

3 participants