kadirozde commented on a change in pull request #915: URL: https://github.com/apache/phoenix/pull/915#discussion_r504917195
########## File path: phoenix-core/src/main/java/org/apache/phoenix/coprocessor/GlobalIndexRegionScanner.java ########## @@ -240,4 +320,1033 @@ protected boolean isColumnIncluded(Cell cell) { byte[] qualifier = CellUtil.cloneQualifier(cell); return set.contains(qualifier); } + @VisibleForTesting + public boolean shouldVerify(IndexTool.IndexVerifyType verifyType, + byte[] indexRowKey, Scan scan, Region region, IndexMaintainer indexMaintainer, + IndexVerificationResultRepository verificationResultRepository, boolean shouldVerifyCheckDone) throws IOException { + this.verifyType = verifyType; + this.indexRowKeyforReadRepair = indexRowKey; + this.scan = scan; + this.region = region; + this.indexMaintainer = indexMaintainer; + this.verificationResultRepository = verificationResultRepository; + this.shouldVerifyCheckDone = shouldVerifyCheckDone; + return shouldVerify(); + } + + protected boolean shouldVerify() throws IOException { + // In case of read repair, proceed with rebuild + // All other types of rebuilds/verification should be incrementally performed if appropriate param is passed + byte[] lastVerifyTimeValue = scan.getAttribute(UngroupedAggregateRegionObserver.INDEX_RETRY_VERIFY); + Long lastVerifyTime = lastVerifyTimeValue == null ? 0 : Bytes.toLong(lastVerifyTimeValue); + if(indexRowKeyforReadRepair != null || lastVerifyTime == 0 || shouldVerifyCheckDone) { + return true; + } + + IndexToolVerificationResult verificationResultTemp = verificationResultRepository + .getVerificationResult(lastVerifyTime, scan, region, indexMaintainer.getIndexTableName()) ; + if(verificationResultTemp != null) { + verificationResult = verificationResultTemp; + } + shouldVerifyCheckDone = true; + return verificationResultTemp == null; + } + + @Override + public HRegionInfo getRegionInfo() { + return region.getRegionInfo(); + } + + @Override + public boolean isFilterDone() { + return false; + } + + private void closeTables() throws IOException { + hTableFactory.shutdown(); + if (indexHTable != null) { + indexHTable.close(); + } + if (dataHTable != null) { + dataHTable.close(); + } + } + @Override + public void close() throws IOException { + innerScanner.close(); + if (indexRowKeyforReadRepair != null) { + closeTables(); + return; + } + if (verify) { + try { + if (verificationResultRepository != null) { + verificationResultRepository.logToIndexToolResultTable(verificationResult, + verifyType, region.getRegionInfo().getRegionName(), skipped); + } + } finally { + this.pool.stop("IndexRegionObserverRegionScanner is closing"); + closeTables(); + if (verificationResultRepository != null) { + verificationResultRepository.close(); + } + if (verificationOutputRepository != null) { + verificationOutputRepository.close(); + } + } + } + else { + this.pool.stop("IndexRegionObserverRegionScanner is closing"); + closeTables(); + } + } + + @VisibleForTesting + public int setIndexTableTTL(int ttl) { + indexTableTTL = ttl; + return 0; Review comment: This method is used only for the unit test. @swaroopak, can you answer the question? ---------------------------------------------------------------- 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: us...@infra.apache.org