rkhachatryan commented on a change in pull request #14052:
URL: https://github.com/apache/flink/pull/14052#discussion_r524331004



##########
File path: 
flink-runtime/src/main/java/org/apache/flink/runtime/io/network/partition/consumer/RemoteInputChannel.java
##########
@@ -481,7 +489,7 @@ private boolean addPriorityBuffer(SequenceBuffer 
sequenceBuffer) throws IOExcept
                if 
(channelStatePersister.checkForBarrier(sequenceBuffer.buffer)) {
                        // checkpoint was not yet started by task thread,
                        // so remember the numbers of buffers to spill for the 
time when it will be started
-                       numBuffersOvertaken = 
receivedBuffers.getNumUnprioritizedElements();
+                       lastOvertakenSequenceNumber = 
sequenceBuffer.sequenceNumber;

Review comment:
       This is addressed in the next commit by adding `lastBarrierId` and 
checking 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.

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


Reply via email to