[GitHub] [flink-kubernetes-operator] morhidi commented on pull request #249: [FLINK-26179] Support for periodic savepoints

2022-05-31 Thread GitBox
morhidi commented on PR #249: URL: https://github.com/apache/flink-kubernetes-operator/pull/249#issuecomment-1142428887 > @morhidi I wasn't planning on changing the history logic, that would also be a breaking change compared to previous versions so it would need transformation logic to ma

[GitHub] [flink-kubernetes-operator] morhidi commented on pull request #249: [FLINK-26179] Support for periodic savepoints

2022-05-31 Thread GitBox
morhidi commented on PR #249: URL: https://github.com/apache/flink-kubernetes-operator/pull/249#issuecomment-1142302043 @gyfora I was comparing the savepoint history with the ones on the dashboard. Does it make sense to display it the same way e.g. latest on top, human readable timestamp?

[GitHub] [flink-kubernetes-operator] morhidi commented on pull request #249: [FLINK-26179] Support for periodic savepoints

2022-05-31 Thread GitBox
morhidi commented on PR #249: URL: https://github.com/apache/flink-kubernetes-operator/pull/249#issuecomment-1142279072 Hitting this locally after a while, doesn't seem to be related to the current change```2022-05-31 17:07:55,479 o.a.f.k.o.o.SavepointObserver [INFO ] [default.basic-check

[GitHub] [flink-kubernetes-operator] morhidi commented on pull request #249: [FLINK-26179] Support for periodic savepoints

2022-05-30 Thread GitBox
morhidi commented on PR #249: URL: https://github.com/apache/flink-kubernetes-operator/pull/249#issuecomment-1141737882 Thanks @gyfora looks promising. I'll have a look later today. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi