PHOENIX-6055: Not matching index mutation error needs to report more …

…information rather than first expected mutation
apache · Sep 22, 2020 · 4c7beb2 · 4c7beb2
1 parent f5cc0ad
commit 4c7beb2
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/phoenix-core/src/main/java/org/apache/phoenix/coprocessor/IndexRebuildRegionScanner.java b/phoenix-core/src/main/java/org/apache/phoenix/coprocessor/IndexRebuildRegionScanner.java
@@ -689,7 +689,7 @@ public boolean verifySingleIndexRow(Result indexRow, IndexToolVerificationResult
         int actualSize = actualMutationList.size();
         Mutation expected = null;
         Mutation previousExpected;
-        Mutation actual;
+        Mutation actual = null;
         while (expectedIndex < expectedSize && actualIndex <actualSize) {
             previousExpected = expected;
             expected = expectedMutationList.get(expectedIndex);
@@ -768,7 +768,8 @@ public boolean verifySingleIndexRow(Result indexRow, IndexToolVerificationResult
                 }
             } else {
                 byte[] dataKey = indexMaintainer.buildDataRowKey(new ImmutableBytesWritable(indexRow.getRow()), viewConstants);
-                String errorMsg = "Not matching index row";
+                String errorMsg = String.format("Not matching index row. matchingCount=0. expectedIndex=%d. expectedMutationSize=%d. actualIndex=%d. actualMutationSize=%d. expectedTs=%d. actualTs=%d",
+                        expectedIndex, expectedSize, actualIndex, actualSize, (expected == null ? 0:getTimestamp(expected)), (actual == null ? 0: getTimestamp(actual)));
                 logToIndexToolOutputTable(dataKey, indexRow.getRow(),
                         getTimestamp(expectedMutationList.get(0)), 0L, errorMsg);
                 verificationPhaseResult.setInvalidIndexRowCount(verificationPhaseResult.getInvalidIndexRowCount() + 1);