kadirozde commented on code in PR #2223:
URL: https://github.com/apache/phoenix/pull/2223#discussion_r2213006013


##########
phoenix-core-server/src/main/java/org/apache/phoenix/index/GlobalIndexChecker.java:
##########
@@ -452,6 +452,7 @@ private void repairIndexRows(byte[] indexRowKey, long ts, 
List<Cell> row) throws
                 // The row is "unverified". Rewind the scanner and let the row 
be scanned again
                 // so that it can be repaired
                 scanner.close();
+                indexScan.withStartRow(CellUtil.cloneRow(row.get(0)), true);

Review Comment:
   The comments starting on line 452 should read as follows
   We got a different index row. This means either the unverified index row has 
been deleted by the single row rebuild as part of the read repair process, or 
the filters on the scan object skip the rebuilt index row.  If this new row is 
"verified" (or empty) then we are good to go.
   
   



-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@phoenix.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to