Fokko commented on issue #1705: URL: https://github.com/apache/iceberg-python/issues/1705#issuecomment-2676359612
Thanks @djouallah for reporting this. I think we should suppress the warning when `Provided Region is None`. There is zero value in that. The message is just to avoid that you start pulling large amounts of data across regions without being aware of it. > By introducing a configuration flag, we allow users (or integration tests) to disable automatic S3 region resolution. As mentioned in https://github.com/apache/iceberg-python/issues/1708 I think we can also explore other options. An important part of testing is that we test the code-paths that are used by the users. Therefore I'm not a fan of disabling this for the integration tests. -- 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
