dimas-b commented on code in PR #3826:
URL: https://github.com/apache/polaris/pull/3826#discussion_r2898552189


##########
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:
   > a well-designed server should not produce multiple configurations for the 
same prefix [...]
   
   Then we should state this in the spec. (I agree)



-- 
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]

Reply via email to