gh-yzou commented on code in PR #3826:
URL: https://github.com/apache/polaris/pull/3826#discussion_r2898517025
##########
spec/polaris-catalog-apis/generic-tables-api.yaml:
##########
@@ -256,6 +264,51 @@ components:
items:
$ref:
'../iceberg-rest-catalog-open-api.yaml#/components/schemas/TableIdentifier'
+
+ StorageAccessConfig:
+ type: object
+ required:
+ - prefix
+ - config
+ properties:
+ prefix:
+ type: string
+ description: Indicates a storage location prefix where the
configuration is relevant. Clients should choose the most
+ specific prefix (by selecting the longest common prefix) if
several configurations of the same type are available.
+ If multiple configurations share the same longest common prefix,
the client may select any of them.
Review Comment:
From my perspective, the server’s responsibility is to ensure that it
produces valid credentials for accessing the table data. As long as the
configurations are valid for accessing files under the prefi, it should not
matter which configuration the client chooses to use.
Specifying whether the client uses the first or the last configuration
matches does not change the server’s responsibility.
Furthermore, a well-designed server should not produce multiple
configurations for the same prefix, as this can be confusing for clients
regardless of which one they select.
--
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]
