You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@flink.apache.org by fa...@apache.org on 2024/01/23 10:09:25 UTC
(flink) 05/05: [hotfix] Remove the wrong `@Nullable` annotation of FailureHandlingResult due to the constructor checks `verticesToRestart` is not null
This is an automated email from the ASF dual-hosted git repository.
fanrui pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/flink.git
commit ad3f13f1d5b1604fda76292944233be78a87b456
Author: Rui Fan <19...@gmail.com>
AuthorDate: Wed Jan 10 16:53:41 2024 +0800
[hotfix] Remove the wrong `@Nullable` annotation of FailureHandlingResult due to the constructor checks `verticesToRestart` is not null
---
.../flink/runtime/executiongraph/failover/FailureHandlingResult.java | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/flink-runtime/src/main/java/org/apache/flink/runtime/executiongraph/failover/FailureHandlingResult.java b/flink-runtime/src/main/java/org/apache/flink/runtime/executiongraph/failover/FailureHandlingResult.java
index 68c1983b0fe..3c3dbc9e335 100644
--- a/flink-runtime/src/main/java/org/apache/flink/runtime/executiongraph/failover/FailureHandlingResult.java
+++ b/flink-runtime/src/main/java/org/apache/flink/runtime/executiongraph/failover/FailureHandlingResult.java
@@ -87,7 +87,7 @@ public class FailureHandlingResult {
@Nullable Throwable cause,
long timestamp,
CompletableFuture<Map<String, String>> failureLabels,
- @Nullable Set<ExecutionVertexID> verticesToRestart,
+ Set<ExecutionVertexID> verticesToRestart,
long restartDelayMS,
boolean globalFailure,
boolean isRootCause) {
@@ -247,7 +247,7 @@ public class FailureHandlingResult {
@Nullable Throwable cause,
long timestamp,
CompletableFuture<Map<String, String>> failureLabels,
- @Nullable Set<ExecutionVertexID> verticesToRestart,
+ Set<ExecutionVertexID> verticesToRestart,
long restartDelayMS,
boolean globalFailure,
boolean isRootCause) {