BTheunissen commented on issue #1104: URL: https://github.com/apache/iceberg-python/issues/1104#issuecomment-2325218390
@kevinjqliu Definitely fair enough that the reason for deprecation being that the catalog settings are generally exposed as a `Dict[str, str]` and the botocore.Session object being passed in breaks this convention. I'd be fine removing if the ticket to track credential refresh was written up, I'd take a crack at implementing it but honestly the workarounds I've had to do to support it for both the Python boto clients, and the underlying filesystem implementations is pretty hacky, there are [some existing issues](https://github.com/apache/arrow/issues/41794) on the same topic open against the Arrow project. -- 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
