wangyang0918 commented on pull request #35:
URL: 
https://github.com/apache/flink-kubernetes-operator/pull/35#issuecomment-1058750206


   For this configuration, maybe we could set the rest service exposed type to 
`ClusterIP` in `FlinkUtils#loadDefaultConfig()` if it is not configured 
explicitly in the ConfigMap `flink-default-config`. Of cause, it could also 
take effect if the user override it via FlinkDeployment 
`.spec.flinkConfiguration`.
   
   
   Separating the service accounts is a great effort. Thanks for doing this.


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