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

chinmayskulkarni pushed a commit to branch phoenix-stats
in repository https://gitbox.apache.org/repos/asf/phoenix.git


The following commit(s) were added to refs/heads/phoenix-stats by this push:
     new b744688  Correct incorrectly formed UTs with wrong sized slotSpan.
b744688 is described below

commit b744688b6123745e0b293ca0dfa2567d56acd883
Author: Daniel <[email protected]>
AuthorDate: Tue May 7 11:42:57 2019 -0700

    Correct incorrectly formed UTs with wrong sized slotSpan.
---
 .../java/org/apache/phoenix/compile/ScanRangesIntersectTest.java    | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git 
a/phoenix-core/src/test/java/org/apache/phoenix/compile/ScanRangesIntersectTest.java
 
b/phoenix-core/src/test/java/org/apache/phoenix/compile/ScanRangesIntersectTest.java
index df1bade..faf7a7d 100644
--- 
a/phoenix-core/src/test/java/org/apache/phoenix/compile/ScanRangesIntersectTest.java
+++ 
b/phoenix-core/src/test/java/org/apache/phoenix/compile/ScanRangesIntersectTest.java
@@ -295,7 +295,6 @@ public class ScanRangesIntersectTest {
     public void getRowKeyRangesTestNotFullyQualifiedRowKeyLookUp() {
         int rowKeySchemaFields = 2;
         RowKeySchema schema = buildSimpleRowKeySchema(rowKeySchemaFields);
-        int[] slotSpan = new int[rowKeySchemaFields];
 
         String keyString1 = "A";
         String keyString2 = "B";
@@ -306,6 +305,8 @@ public class ScanRangesIntersectTest {
         List<List<KeyRange>> ranges = new ArrayList<>();
         ranges.add(Lists.newArrayList(rangeKeyRange1, rangeKeyRange2));
 
+        int[] slotSpan = new int[ranges.size()];
+
         ScanRanges scanRanges = ScanRanges.create(schema, ranges, slotSpan, 
null, true, -1);
 
         List<KeyRange> rowKeyRanges = scanRanges.getRowKeyRanges();
@@ -401,7 +402,6 @@ public class ScanRangesIntersectTest {
     public void getRowKeyRangesAdjacentSubRanges() {
         int rowKeySchemaFields = 2;
         RowKeySchema schema = buildSimpleRowKeySchema(rowKeySchemaFields);
-        int[] slotSpan = new int[rowKeySchemaFields];
 
         List<KeyRange> keyRanges = new ArrayList<>();
         keyRanges.add(KeyRange.getKeyRange(stringToByteArray("A"), true,
@@ -416,6 +416,8 @@ public class ScanRangesIntersectTest {
         List<List<KeyRange>> ranges = new ArrayList<>();
         ranges.add(keyRanges);
 
+        int[] slotSpan = new int[ranges.size()];
+
         ScanRanges scanRanges = ScanRanges.create(schema, ranges, slotSpan, 
null, true, -1);
 
         List<KeyRange> rowKeyRanges = scanRanges.getRowKeyRanges();

Reply via email to