You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@phoenix.apache.org by mi...@apache.org on 2019/10/10 09:21:00 UTC

[phoenix] branch master updated: PHOENIX-5493 Remove unnecesary iteration in BaseResultIterator

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

mihir6692 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/phoenix.git


The following commit(s) were added to refs/heads/master by this push:
     new bf30a40  PHOENIX-5493 Remove unnecesary iteration in BaseResultIterator
bf30a40 is described below

commit bf30a40006e15b60eb140c974dc4317e4e83de75
Author: Aman Poonia <ap...@salesforce.com>
AuthorDate: Thu Oct 10 14:50:33 2019 +0530

    PHOENIX-5493 Remove unnecesary iteration in BaseResultIterator
---
 .../src/main/java/org/apache/phoenix/iterate/BaseResultIterators.java    | 1 +
 1 file changed, 1 insertion(+)

diff --git a/phoenix-core/src/main/java/org/apache/phoenix/iterate/BaseResultIterators.java b/phoenix-core/src/main/java/org/apache/phoenix/iterate/BaseResultIterators.java
index c9899a9..8fd368a 100644
--- a/phoenix-core/src/main/java/org/apache/phoenix/iterate/BaseResultIterators.java
+++ b/phoenix-core/src/main/java/org/apache/phoenix/iterate/BaseResultIterators.java
@@ -777,6 +777,7 @@ public abstract class BaseResultIterators extends ExplainTable implements Result
             for (KeyRange range : ranges) {
                 if (!range.isSingleKey()) {
                     hasRange = true;
+                    break;
                 }
             }
             slotSpan[offset] = rangeSpan - 1;