sarthaksin1857 commented on code in PR #14966:
URL: https://github.com/apache/iceberg/pull/14966#discussion_r2721542536
##########
azure/src/main/java/org/apache/iceberg/azure/adlsv2/ADLSFileIO.java:
##########
@@ -109,6 +131,16 @@ public DataLakeFileSystemClient client(String path) {
@VisibleForTesting
DataLakeFileSystemClient client(ADLSLocation location) {
+ if (clientSupplier != null) {
Review Comment:
The main difference here an aws/gcp is that one client can be used for
multiple locations/buckets. In azure the client is tied to a certain
container/path. I will explore creating a client with A Function<Location,
ADLSClient> so that we can specify multiple containers with this class
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
