HonahX commented on code in PR #1277:
URL: https://github.com/apache/polaris/pull/1277#discussion_r2031808574
##########
spec/generated/bundled-polaris-catalog-service.yaml:
##########
@@ -4005,6 +4005,23 @@ components:
properties:
target:
$ref: '#/components/schemas/PolicyAttachmentTarget'
+ 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
+ - **namespace:** A list representing the hierarchical parent path
to the policy, ordered from higher level namespace to lower.
+ required:
+ - inherited
+ - namespace
Review Comment:
We are re-using the fields of `#/components/schemas/Policy` here, so the
overall structure of `ApplicablePolicy` is
- inherited
- namespace
- policy-type
- inheritable
- name
- description
- content
- version
Those that are required in `Policy` will also be required in
`ApplicablePolicy`. We do not need to explicitly write them here. I've checked
the generated class to verify 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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
