Github user zentol commented on a diff in the pull request:

    https://github.com/apache/flink/pull/5893#discussion_r183632835
  
    --- Diff: 
flink-runtime/src/main/java/org/apache/flink/runtime/rest/handler/legacy/SubtasksAllAccumulatorsHandler.java
 ---
    @@ -97,15 +97,14 @@ public static String 
createSubtasksAccumulatorsJson(AccessExecutionJobVertex job
     
                gen.writeArrayFieldStart("subtasks");
     
    -           int num = 0;
                for (AccessExecutionVertex vertex : 
jobVertex.getTaskVertices()) {
     
                        TaskManagerLocation location = 
vertex.getCurrentAssignedResourceLocation();
                        String locationString = location == null ? 
"(unassigned)" : location.getHostname();
     
                        gen.writeStartObject();
     
    -                   gen.writeNumberField("subtask", num++);
    +                   gen.writeNumberField("subtask", 
vertex.getCurrentExecutionAttempt().getParallelSubtaskIndex());
    --- End diff --
    
    Let's keep this at it is.


---

Reply via email to