pltbkd commented on code in PR #20733:
URL: https://github.com/apache/flink/pull/20733#discussion_r963443434


##########
flink-runtime/src/main/java/org/apache/flink/runtime/messages/webmonitor/JobDetails.java:
##########
@@ -411,7 +382,26 @@ public JobDetails deserialize(
                     lastUpdateTime,
                     numVerticesPerExecutionState,
                     numTasks,
-                    attempts);
+                    new HashMap<>());
+        }
+    }
+
+    public static final class CurrentAttempts implements Serializable {
+        private final int representativeAttempt;
+
+        private final List<Integer> currentAttempts;

Review Comment:
   I've been thinking about this but I was wondering if it's too customized for 
this pr. But maybe it's ok. I'll modify here.



##########
flink-runtime/src/main/java/org/apache/flink/runtime/messages/webmonitor/JobDetails.java:
##########
@@ -411,7 +382,26 @@ public JobDetails deserialize(
                     lastUpdateTime,
                     numVerticesPerExecutionState,
                     numTasks,
-                    attempts);
+                    new HashMap<>());
+        }
+    }
+
+    public static final class CurrentAttempts implements Serializable {

Review Comment:
   Sorry for the mistake. I'll add it.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to