gokceni commented on a change in pull request #891:
URL: https://github.com/apache/phoenix/pull/891#discussion_r493745610
##########
File path:
phoenix-core/src/main/java/org/apache/phoenix/coprocessor/IndexRebuildRegionScanner.java
##########
@@ -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",
Review comment:
@kadirozde Rebased. I don't think we put the actualTs since in the next
line we always put 0L. For the expected, since we put expectedMutations.get(0)
I don't think it hurts to put what it is here in case we change the above loop
and since we put the actualTs. what do you think?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
