tweise commented on a change in pull request #6:
URL: 
https://github.com/apache/flink-kubernetes-operator/pull/6#discussion_r809465504



##########
File path: helm/flink-operator/templates/flink-operator.yaml
##########
@@ -118,6 +124,9 @@ metadata:
   labels:
     {{- include "flink-operator.labels" . | nindent 4 }}
 data:
+  flink-conf.yaml: |+

Review comment:
       I think I prefer the logger since every infrastructure have their unique 
metric backend requirements and it is good to "see" something out of the box. 
Maybe we can include the the prometheus configs here as a comment so that it's 
easy to activate?




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