[GitHub] [flink] SinBex commented on a diff in pull request #21695: [FLINK-30707][runtime] Let speculative execution take input data amount into account when detecting slow tasks.

[GitBox]

SinBex commented on code in PR #21695:
URL: https://github.com/apache/flink/pull/21695#discussion_r1081130145


##########
flink-runtime/src/main/java/org/apache/flink/runtime/scheduler/adaptivebatch/PointwiseBlockingResultInfo.java:
##########
@@ -60,4 +61,23 @@ public long getNumBytesProduced() {
                 .flatMapToLong(Arrays::stream)
                 .reduce(0L, Long::sum);
     }
+
+    @Override
+    public long getExecutionVertexInputNumBytes(
+            IndexRange partitionIndexRange, IndexRange subpartitionIndexRange) 
{
+        checkState(
+                subpartitionBytesByPartitionIndex.size() > 
partitionIndexRange.getEndIndex(),
+                "Not all partition infos are ready");

Review Comment:
   Thanks for reminding, I have added the check.



-- 
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: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org