[GitHub] [flink-kubernetes-operator] gyfora commented on a diff in pull request #252: [FLINK-27889] fix: Catch the error when last reconciled spec is null

2022-06-04 Thread GitBox
gyfora commented on code in PR #252: URL: https://github.com/apache/flink-kubernetes-operator/pull/252#discussion_r889595076 ## flink-kubernetes-operator/src/main/java/org/apache/flink/kubernetes/operator/service/FlinkService.java: ## @@ -229,14 +229,12 @@ private

[GitHub] [flink-kubernetes-operator] gyfora commented on a diff in pull request #252: [FLINK-27889] fix: Catch the error when last reconciled spec is null

2022-06-04 Thread GitBox
gyfora commented on code in PR #252: URL: https://github.com/apache/flink-kubernetes-operator/pull/252#discussion_r889524742 ## flink-kubernetes-operator/src/test/java/org/apache/flink/kubernetes/operator/controller/FlinkDeploymentControllerTest.java: ## @@ -274,6 +279,15 @@

[GitHub] [flink-kubernetes-operator] gyfora commented on a diff in pull request #252: [FLINK-27889] fix: Catch the error when last reconciled spec is null

2022-06-04 Thread GitBox
gyfora commented on code in PR #252: URL: https://github.com/apache/flink-kubernetes-operator/pull/252#discussion_r88952 ## flink-kubernetes-operator/src/test/java/org/apache/flink/kubernetes/operator/controller/FlinkDeploymentControllerTest.java: ## @@ -274,6 +279,15 @@

[GitHub] [flink-kubernetes-operator] gyfora commented on a diff in pull request #252: [FLINK-27889] fix: Catch the error when last reconciled spec is null

2022-06-04 Thread GitBox
gyfora commented on code in PR #252: URL: https://github.com/apache/flink-kubernetes-operator/pull/252#discussion_r889505556 ## flink-kubernetes-operator/src/main/java/org/apache/flink/kubernetes/operator/config/FlinkConfigManager.java: ## @@ -134,6 +135,11 @@ public

[GitHub] [flink-kubernetes-operator] gyfora commented on a diff in pull request #252: [FLINK-27889] fix: Catch the error when last reconciled spec is null

2022-06-03 Thread GitBox
gyfora commented on code in PR #252: URL: https://github.com/apache/flink-kubernetes-operator/pull/252#discussion_r889333575 ## flink-kubernetes-operator/src/test/java/org/apache/flink/kubernetes/operator/controller/FlinkDeploymentControllerTest.java: ## @@ -274,6 +278,89 @@

[GitHub] [flink-kubernetes-operator] gyfora commented on a diff in pull request #252: [FLINK-27889] fix: Catch the error when last reconciled spec is null

2022-06-03 Thread GitBox
gyfora commented on code in PR #252: URL: https://github.com/apache/flink-kubernetes-operator/pull/252#discussion_r889333575 ## flink-kubernetes-operator/src/test/java/org/apache/flink/kubernetes/operator/controller/FlinkDeploymentControllerTest.java: ## @@ -274,6 +278,89 @@

[GitHub] [flink-kubernetes-operator] gyfora commented on a diff in pull request #252: [FLINK-27889] fix: Catch the error when last reconciled spec is null

2022-06-03 Thread GitBox
gyfora commented on code in PR #252: URL: https://github.com/apache/flink-kubernetes-operator/pull/252#discussion_r888767237 ## flink-kubernetes-operator/src/main/java/org/apache/flink/kubernetes/operator/service/FlinkService.java: ## @@ -354,8 +355,54 @@ protected

[GitHub] [flink-kubernetes-operator] gyfora commented on a diff in pull request #252: [FLINK-27889] fix: Catch the error when last reconciled spec is null

2022-06-03 Thread GitBox
gyfora commented on code in PR #252: URL: https://github.com/apache/flink-kubernetes-operator/pull/252#discussion_r888762251 ## flink-kubernetes-operator/src/main/java/org/apache/flink/kubernetes/operator/exception/LastReconciledSpecException.java: ## @@ -0,0 +1,25 @@ +/* + *

[GitHub] [flink-kubernetes-operator] gyfora commented on a diff in pull request #252: [FLINK-27889] fix: Catch the error when last reconciled spec is null

2022-06-03 Thread GitBox
gyfora commented on code in PR #252: URL: https://github.com/apache/flink-kubernetes-operator/pull/252#discussion_r888760743 ## flink-kubernetes-operator/src/main/java/org/apache/flink/kubernetes/operator/service/FlinkService.java: ## @@ -354,8 +355,54 @@ protected