usamj commented on PR #357: URL: https://github.com/apache/flink-kubernetes-operator/pull/357#issuecomment-1232281033
I agree with @gyfora. and aligns with what I originally thought when I saw the change. I don't think there is enough benefit to the JM being a StatefulSet My other concern is whether we should only use StatefulSets for the TM's or whether using StatefulSets/Deployment for them should be a configurable option. The reason for this is that recovery time for a TM failing would be longer if it used StatefulSets. It could also result in issues and user action to fix if a TM pod was stuck in a Terminating state which could be unnecessary pain for users if they aren't using PVC's with their TM's -- 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
