jiakai-li commented on code in PR #1453: URL: https://github.com/apache/iceberg-python/pull/1453#discussion_r1896132680
########## pyiceberg/io/pyarrow.py: ########## @@ -377,6 +377,12 @@ def _initialize_fs(self, scheme: str, netloc: Optional[str] = None) -> FileSyste if force_virtual_addressing := self.properties.get(S3_FORCE_VIRTUAL_ADDRESSING): client_kwargs["force_virtual_addressing"] = property_as_bool(self.properties, force_virtual_addressing, False) + # Override the default s3.region if netloc(bucket) resolves to a different region Review Comment: Thank you @kevinjqliu ! Can I please have some more guidance regards what do you mean by closer to where `regions` is set? Do you mean right below where we set the `client_kwargs` in the function? Thanks! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org