gh-yzou commented on code in PR #3826:
URL: https://github.com/apache/polaris/pull/3826#discussion_r2898215697


##########
spec/polaris-catalog-apis/generic-tables-api.yaml:
##########
@@ -256,6 +264,53 @@ 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 prefix) if several 
configurations of the same type are available.

Review Comment:
   If there are configurations for the same prefix, i think client should be 
free to choose any of them, given that they are all valid. I felt explicitly 
stating the first one is over restricted to me. 
   
   Added more clarification for such case.



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