zhuzhurk commented on a change in pull request #11855:
URL: https://github.com/apache/flink/pull/11855#discussion_r412816166
##########
File path:
flink-runtime/src/main/java/org/apache/flink/runtime/jobgraph/IntermediateResultPartitionID.java
##########
@@ -27,29 +28,61 @@
/**
* Id identifying {@link IntermediateResultPartition}.
*/
-public class IntermediateResultPartitionID extends AbstractID implements
ResultID {
+public class IntermediateResultPartitionID implements ResultID {
private static final long serialVersionUID = 1L;
+ private final IntermediateDataSetID intermediateDataSetID;
+ private final int partitionNum;
+
/**
- * Creates an new random intermediate result partition ID.
+ * Creates an new random intermediate result partition ID for testing.
*/
+ @VisibleForTesting
public IntermediateResultPartitionID() {
- super();
+ this.partitionNum = -1;
+ this.intermediateDataSetID = new IntermediateDataSetID(new
AbstractID());
Review comment:
`IntermediateDataSetID` has a param-less ctor I think.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
