You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@beam.apache.org by ud...@apache.org on 2020/01/10 20:54:15 UTC

[beam] 01/01: [BEAM-4287] Fix to use the residual instead of the current restriction on process continuations.

This is an automated email from the ASF dual-hosted git repository.

udim pushed a commit to branch release-2.18.0
in repository https://gitbox.apache.org/repos/asf/beam.git

commit 9e58be17043c9dcbd36dc5e48939f358d877c366
Author: Luke Cwik <lc...@google.com>
AuthorDate: Thu Dec 5 13:42:16 2019 -0800

    [BEAM-4287] Fix to use the residual instead of the current restriction on process continuations.
    
    This is a forward fix for https://github.com/apache/beam/pull/10258 to fix Dataflow VR postcommit suite.
---
 .../beam/runners/core/construction/SplittableParDoNaiveBounded.java     | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/runners/core-construction-java/src/main/java/org/apache/beam/runners/core/construction/SplittableParDoNaiveBounded.java b/runners/core-construction-java/src/main/java/org/apache/beam/runners/core/construction/SplittableParDoNaiveBounded.java
index d975b4f..8a41d7e 100644
--- a/runners/core-construction-java/src/main/java/org/apache/beam/runners/core/construction/SplittableParDoNaiveBounded.java
+++ b/runners/core-construction-java/src/main/java/org/apache/beam/runners/core/construction/SplittableParDoNaiveBounded.java
@@ -147,7 +147,7 @@ public class SplittableParDoNaiveBounded {
         ProcessContinuation continuation =
             invoker.invokeProcessElement(new NestedProcessContext<>(fn, c, element, w, tracker));
         if (continuation.shouldResume()) {
-          restriction = tracker.currentRestriction();
+          restriction = tracker.trySplit(0).getResidual();
           Uninterruptibles.sleepUninterruptibly(
               continuation.resumeDelay().getMillis(), TimeUnit.MILLISECONDS);
         } else {