Fix bounds check in MultiChannelGroupByHash

There's an off-by-one error in the check that can cause a failure when the page is empty
prestodb · Mar 27, 2018 · 0095d1d · 0095d1d
1 parent 84be87e
commit 0095d1d
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/presto-main/src/main/java/com/facebook/presto/operator/MultiChannelGroupByHash.java b/presto-main/src/main/java/com/facebook/presto/operator/MultiChannelGroupByHash.java
@@ -632,7 +632,7 @@ public GetNonDictionaryGroupIdsWork(Page page)
         public boolean process()
         {
             int positionCount = page.getPositionCount();
-            checkState(lastPosition < positionCount, "position count out of bound");
+            checkState(lastPosition <= positionCount, "position count out of bound");
             checkState(!finished);
 
             // needRehash() == false indicates we have reached capacity boundary and a rehash is needed.