apache#51 [euphoria-core] Skip processing empty window merge sets

seznam · Jul 9, 2018 · b2a35e7 · b2a35e7
1 parent 3ae4e69
commit b2a35e7
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/...ia/euphoria-core/src/main/java/cz/seznam/euphoria/core/executor/greduce/GroupReducer.java b/...ia/euphoria-core/src/main/java/cz/seznam/euphoria/core/executor/greduce/GroupReducer.java
@@ -192,7 +192,11 @@ private Pair<WID, Trigger.TriggerResult> mergeWindows(WID newWindow) {
       //  c) the target window's trigger #onClear won't be called
       sources.remove(target);
 
-      // XXX only if sources non empty!
+      // ~ do not bother with the rest of thi for loop if we have
+      // no source windows to merge
+      if (sources.isEmpty()) {
+        continue;
+      }
 
       // ~ make sure to create the target state if necessary
       State targetState = getStateForUpdate(target);