Skip to content
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.

Sign up for GitHub

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

Jump to bottom

feat: Improvements to the S3 API #6028

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

feat: Improvements to the S3 API #6028

wants to merge 6 commits into from

Conversation

malhotrashivam
Copy link
Contributor

@malhotrashivam malhotrashivam commented Sep 6, 2024 •
edited
Loading

Closes #6016, #6017

  • If region_name is not set, it is derived internally at the cost of one additional request
  • Fallback to anonymous credentials if default credentials fails

TODO: Add documentation update

@malhotrashivam malhotrashivam added parquet Related to the Parquet integration DocumentationNeeded ReleaseNotesNeeded Release notes are needed s3 labels Sep 6, 2024
@malhotrashivam malhotrashivam added this to the 0.37.0 milestone Sep 6, 2024
@malhotrashivam malhotrashivam self-assigned this Sep 6, 2024
@malhotrashivam malhotrashivam marked this pull request as draft September 6, 2024 18:26
@malhotrashivam malhotrashivam changed the title feat: [DO NOT MERGE] Improvements to the S3 API feat: Improvements to the S3 API Sep 6, 2024
@malhotrashivam malhotrashivam marked this pull request as ready for review September 16, 2024 20:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@devinrsmith devinrsmith devinrsmith approved these changes

@chipkent chipkent Awaiting requested review from chipkent chipkent is a code owner

@jmao-denver jmao-denver Awaiting requested review from jmao-denver jmao-denver is a code owner

@rcaudy rcaudy Awaiting requested review from rcaudy rcaudy is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@malhotrashivam malhotrashivam

Labels
DocumentationNeeded parquet Related to the Parquet integration ReleaseNotesNeeded Release notes are needed s3
Projects
None yet
Milestone
0.37.0
Development

Successfully merging this pull request may close these issues.

Enable S3 to work without explicit region_name
3 participants
@malhotrashivam @chipkent @devinrsmith