-
Notifications
You must be signed in to change notification settings - Fork 3k
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.
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
Document accessing multiple azure storage accounts in single catalog #23716
Document accessing multiple azure storage accounts in single catalog #23716
Conversation
Also .. just to be sure .. this page is generic for all table formats.. so the assumption is that the approach works for all of them. If not we need to clarify |
5f05da9
to
1f05450
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mosabua addressed comments. Also added a bit more details. please re-review
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry.. more suggestions .. nearly there.
1f05450
to
9ffb641
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One nit and a final update of the commit message please .. its too long .. use a shorter title and if needed a paragraph after, then the PR is good to go
https://trino.io/development/process#pull-request-and-commit-guidelines-
9ffb641
to
ba52a0a
Compare
Description
For #23538
Additional context and related issues
Release notes
( ) This is not user-visible or is docs only, and no release notes are required.
( ) Release notes are required. Please propose a release note for me.
( ) Release notes are required, with the following suggested text: