mchades commented on code in PR #14520:
URL: https://github.com/apache/gravitino/pull/14520#discussion_r4462544878
##########
docs/open-api/policies.yaml:
##########
@@ -894,7 +894,10 @@ components:
nullable: true
policyType:
type: string
- description: The type of the policy
+ description: >-
+ The type of the policy. Known values are matched
case-insensitively; the server
Review Comment:
Adding the uppercase aliases fixes the all-uppercase example, but mixed-case
input is still rejected by the request schemas. `System_Iceberg_Compaction`
with `{"maxPartitionNum":5}` returns 200 on create/update, yet fails both
top-level schemas. This spelling is already documented and covered by the DTO
tests. Please make the request constraints and type selection accept arbitrary
casing, and add payload-validation tests for lowercase, uppercase, and
mixed-case forms; `:docs:build` alone does not check this behavior.
--
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]
