stevenzwu commented on issue #6715: URL: https://github.com/apache/iceberg/issues/6715#issuecomment-1412839741
> I thought when we introduced the config to switch between url and Apache client That config switch code in `AwsProperties` introduced the problem of requiring both url and apache client jars, which then confused `ClasspathSdkHttpServiceProvider` for `WebIdentityTokenFileCredentialsProvider`. @jackye1995 and others, what do you think of the proposal of using reflection to avoid the runtime requirements of both jars, while still supporting the switch config? It would be good to agree on the direction first. Then we can figure out who can pick it up. I will look into my availability. -- 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
