From 5c1f82161df3c83409bfe56560c9b6ae55156227 Mon Sep 17 00:00:00 2001 From: Robert Stupp Date: Thu, 18 Jul 2024 22:13:23 +0200 Subject: [PATCH] Catalog/ADLS: change 'ping' endpoint (#9134) Use `getProperties` instead of `getAccessPolicy`, because latter may require more privileges. --- CHANGELOG.md | 1 + .../java/org/projectnessie/catalog/files/adls/AdlsObjectIO.java | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 83dcc93cf71..e785d48f4c4 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -37,6 +37,7 @@ as necessary. Empty sections will not end in the release notes. - GC: Fix handling of broken manifest files written by pyiceberg up to 0.6.1 - Catalog/ADLS: Don't let endpoint default to warehouse/object-store URI - Catalog/ADLS: More informative error message if mandatory `endpoint` is missing. +- Catalog/ADLS: Use a less restrictive endpoint in the 'ObjectIO.ping' function used for health checks. ### Commits diff --git a/catalog/files/impl/src/main/java/org/projectnessie/catalog/files/adls/AdlsObjectIO.java b/catalog/files/impl/src/main/java/org/projectnessie/catalog/files/adls/AdlsObjectIO.java index 2d178040b2b..95f687dacef 100644 --- a/catalog/files/impl/src/main/java/org/projectnessie/catalog/files/adls/AdlsObjectIO.java +++ b/catalog/files/impl/src/main/java/org/projectnessie/catalog/files/adls/AdlsObjectIO.java @@ -45,7 +45,7 @@ public void ping(StorageUri uri) { AdlsLocation location = adlsLocation(uri); DataLakeFileSystemClient fileSystem = clientSupplier.fileSystemClient(location); - fileSystem.getAccessPolicy(); + fileSystem.getProperties(); } @Override