mchades commented on code in PR #7531:
URL: https://github.com/apache/gravitino/pull/7531#discussion_r2192100164
##########
api/src/main/java/org/apache/gravitino/policy/PolicyOperations.java:
##########
@@ -67,11 +69,18 @@ public interface PolicyOperations {
* @param comment The comment of the policy.
* @param enabled Whether the policy is enabled or not.
* @param content The content of the policy.
+ * @param jobTemplatesToAssociate The list of job templates to associate
with the policy, can be
+ * empty or null if no job templates associated.
* @return The created policy.
* @throws PolicyAlreadyExistsException If the policy already exists.
*/
Policy createPolicy(
- String name, String type, String comment, boolean enabled,
Policy.Content content)
+ String name,
+ String type,
+ String comment,
+ boolean enabled,
+ Policy.Content content,
+ List<JobTemplate> jobTemplatesToAssociate)
Review Comment:
> My point is to make the policy creation success, like what we do for tag
association.
It should align with `createXxxWithTag`, which we do not support yet.
Perhaps we can first retain the original policy creation method, and add the
method with templates later when the requirements are clearer?
--
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]
