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 <1996fan...@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) {