gh-yzou commented on code in PR #3826:
URL: https://github.com/apache/polaris/pull/3826#discussion_r2934102282
##########
spec/polaris-catalog-apis/generic-tables-api.yaml:
##########
@@ -256,6 +264,52 @@ 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 matching prefix,
the client may select any of them. However,
+ a well-designed server should avoid generating multiple
configurations for the same prefix.
Review Comment:
I made this change because it's a minor wording update and doesn't conflict
with any of the existing comments.
For larger changes, I agree that we should definitely call them out.
However, for small adjustments such as wording improvements, we should remain
flexible; otherwise, the process can become unnecessarily tedious.
--
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]
