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.

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

[BUG] Manually specify region in tutorial read_json #1608

Merged
merged 1 commit into from
Nov 15, 2023

Conversation

jaychia
Copy link
Contributor

@jaychia jaychia commented Nov 14, 2023

The change in #1592 introduced a bug where the inferred PyArrow S3FileSystem doesn't correctly infer the region of the bucket

To work-around this, we can manually specify the region_name in our IOConfig. This problem should be less common once we move towards all-native reads.

@jaychia jaychia enabled auto-merge (squash) November 14, 2023 21:44
@github-actions github-actions bot added bug Something isn't working documentation Improvements or additions to documentation labels Nov 14, 2023
Copy link

codecov bot commented Nov 14, 2023

Codecov Report

Merging #1608 (353d1a3) into main (6abc006) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1608   +/-   ##
=======================================
  Coverage   84.65%   84.65%           
=======================================
  Files          54       54           
  Lines        5201     5201           
=======================================
  Hits         4403     4403           
  Misses        798      798           

@jaychia jaychia disabled auto-merge November 15, 2023 18:36
@jaychia jaychia merged commit c4b498a into main Nov 15, 2023
33 of 37 checks passed
@jaychia jaychia deleted the jay/update-notebook-region-name-json-reads branch November 15, 2023 18:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant