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