lasdf1234 opened a new pull request, #14349:
URL: https://github.com/apache/gravitino/pull/14349

   ### What changes were proposed in this pull request?
   
   Validate `jobOptions` for the four Iceberg maintenance policy types when
   `timeTrigger` is set or `commitTrigger` is true, requiring catalog
   type/uri/warehouse (with short aliases). Move orphan `commitTrigger`
   rejection before the shared check so its message still wins.
   
   ### Why are the changes needed?
   
   Incomplete `jobOptions` were accepted at create/alter, then cron submit
   failed with missing `warehouse_location`. Catch this at policy write time.
   
   Fixes #14348
   
   ### Does this PR introduce _any_ user-facing change?
   
   Yes. Create/alter of scheduled Iceberg maintenance policies now rejects
   missing required `jobOptions` keys listed above.
   
   ### How was this patch tested?
   
   `./gradlew :api:test --tests 
org.apache.gravitino.policy.TestExecutablePolicyJobOptionsValidation`
   


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