HonahX commented on code in PR #1277:
URL: https://github.com/apache/polaris/pull/1277#discussion_r2029495147
##########
spec/polaris-catalog-apis/policy-apis.yaml:
##########
@@ -529,6 +529,28 @@ components:
version:
type: integer
+ ApplicablePolicy:
+ allOf:
+ - type: object
+ description: |
+ For policies returned by GetApplicablePolicies, there are 2
additional fields
+
+ - **inherited:** A boolean flag indicating whether the policy is
inherited from target's parents
+ - **namespaces:** A list representing the hierarchical parent path
to the policy, ordered from higher level namespace to lower.
+ required:
+ - inherited
+ - namespaces
+ properties:
+ inherited:
+ type: boolean
+ namespaces:
Review Comment:
Yeah, initially I make it as namespaces because I want to differentiate it
from the `namespace` parameter which is a single string, instead of an array of
string like here. However, from a definition standpoint, namespace is more
accurate since it represents one specific namespace, albeit possibly nested. I
will update it.
--
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: issues-unsubscr...@polaris.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
