mchades commented on code in PR #7531:
URL: https://github.com/apache/gravitino/pull/7531#discussion_r2191337490
##########
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:
here are two questions about this method:
1. What if the job templates to associate do not exist?
2. What if the name of the associated job template changed?
--
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]
