tagarr commented on PR #733: URL: https://github.com/apache/flink-kubernetes-operator/pull/733#issuecomment-1855602604
Looks good. My only question would be to the committers. 1. Would we like the sessionjob changes (i.e. modify labels) be done in flink-kubernetes-webhook/src/main/java/org/apache/flink/kubernetes/operator/admission/mutator/FlinkMutator.java as it currently is or pushed down to the default mutator ? 2. Do we want to add sessionjob mutation for Update as well as for Create operations ? -- 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: issues-unsubscr...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org