[GitHub] [flink-kubernetes-operator] Aitozi commented on pull request #176: [FLINK-27279] Extract common status interfaces

2022-04-22 Thread GitBox
Aitozi commented on PR #176: URL: https://github.com/apache/flink-kubernetes-operator/pull/176#issuecomment-1106331702 Due to the java type erasure, I can't not completly eliminate the `FlinkDeploymentReconciliationStatus` and `FlinkSessionJobReconciliationStatus` because I can't get the

[GitHub] [flink-kubernetes-operator] Aitozi commented on pull request #176: [FLINK-27279] Extract common status interfaces

2022-04-21 Thread GitBox
Aitozi commented on PR #176: URL: https://github.com/apache/flink-kubernetes-operator/pull/176#issuecomment-1104940967 @gyfora Thanks for your suggestion, I will try as you suggested -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [flink-kubernetes-operator] Aitozi commented on pull request #176: [FLINK-27279] Extract common status interfaces

2022-04-20 Thread GitBox
Aitozi commented on PR #176: URL: https://github.com/apache/flink-kubernetes-operator/pull/176#issuecomment-1104562068 I also try to make the ReconcileUtils to a new Object to handle the `reconciledSucceed`, `reconcileError` and `savepointSucceed`. But I have to pass the object everywhere