dimas-b commented on code in PR #3409:
URL: https://github.com/apache/polaris/pull/3409#discussion_r2756422759
##########
polaris-core/src/main/java/org/apache/polaris/core/storage/PolarisStorageConfigurationInfo.java:
##########
@@ -105,6 +107,17 @@ public String serialize() {
}
}
+ public Optional<String> resolveStorageName() {
+ if (getAllowedLocations().isEmpty()) {
+ return Optional.empty();
+ }
+ try {
+ return Optional.ofNullable(new
URI(getAllowedLocations().get(0)).getHost());
Review Comment:
Re: 2) We'd have to expose the location name in Storage Config in the
Management REST API. This is possible, but I'm not sure it's the best path
forward now that the Secrets Manager discussion and per-table storage config
discussion are active and it's not easy to predict how they are going to affect
Storage Config definitions.
https://lists.apache.org/thread/2y2vv8jrgps7s3sr241k6tgm440jzbp9
https://lists.apache.org/thread/boqdzdtmhhk1bncv2xr43sz4nsrhgwro
--
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]
