rkhachatryan commented on a change in pull request #11534:
[FLINK-16537][network] Implement ResultPartition state recovery for unaligned
checkpoint
URL: https://github.com/apache/flink/pull/11534#discussion_r400463370
##########
File path:
flink-runtime/src/main/java/org/apache/flink/runtime/io/network/partition/ResultPartition.java
##########
@@ -107,6 +108,8 @@
@Nullable
protected final BufferCompressor bufferCompressor;
+ private final ChannelStateReader channelStateReader;
+
Review comment:
The same as for `PipelinedSubpartition`: I think it's better to pass
`channelStateReader` as a parameter to `initializeState` (to reduce coupling
and other benefits).
----------------------------------------------------------------
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]
With regards,
Apache Git Services
