Source:
<code>private void filterIndexCheckerMutations(Map<TableInfo, List<Mutation>> mutationMap, Map<TableInfo, List<Mutation>> unverifiedIndexMutations, Map<TableInfo, List<Mutation>> verifiedOrDeletedIndexMutations) throws SQLException { Iterator<Entry<TableInfo, List<Mutation>>> mapIter = mutationMap.entrySet().iterator(); while (mapIter.hasNext()) { Entry<TableInfo, List<Mutation>> pair = mapIter.next(); TableInfo tableInfo = pair.getKey(); if (IndexUtil.isGlobalIndexCheckerEnabled(connection, tableInfo.getHTableName())) { PTable table = tableInfo.getOrigTableRef().getTable(); byte[] emptyCF = SchemaUtil.getEmptyColumnFamily(table); byte[] emptyCQ = EncodedColumnsUtil.getEmptyKeyValueInfo(table).getFirst(); List<Mutation> mutations = pair.getValue(); if (mutations.size() == 0) { continue; } for (Mutation m : mutations) { if (m instanceof Delete) { Put put = new Put(m.getRow()); put.addColumn(emptyCF, emptyCQ, m.getTimestamp(), IndexRegionObserver.UNVERIFIED_BYTES); addToMap(unverifiedIndexMutations, tableInfo, put); <START> addToMap(verifiedOrDeletedIndexMutations, tableInfo, m); <END> } else { addToMap(unverifiedIndexMutations, tableInfo, m); Put put = new Put(m.getRow()); put.addColumn(emptyCF, emptyCQ, m.getTimestamp(), IndexRegionObserver.VERIFIED_BYTES); addToMap(verifiedOrDeletedIndexMutations, tableInfo, put); } } mapIter.remove(); } } }</code><technical_language>Nit: "We need .." sounds is done else. rephrase "set verified bytes for phase3"</technical_language>

Target:
private void filterIndexCheckerMutations(Map<TableInfo, List<Mutation>> mutationMap, Map<TableInfo, List<Mutation>> unverifiedIndexMutations, Map<TableInfo, List<Mutation>> verifiedOrDeletedIndexMutations) throws SQLException { Iterator<Entry<TableInfo, List<Mutation>>> mapIter = mutationMap.entrySet().iterator(); while (mapIter.hasNext()) { Entry<TableInfo, List<Mutation>> pair = mapIter.next(); TableInfo tableInfo = pair.getKey(); if (IndexUtil.isGlobalIndexCheckerEnabled(connection, tableInfo.getHTableName())) { PTable table = tableInfo.getOrigTableRef().getTable(); byte[] emptyCF = SchemaUtil.getEmptyColumnFamily(table); byte[] emptyCQ = EncodedColumnsUtil.getEmptyKeyValueInfo(table).getFirst(); List<Mutation> mutations = pair.getValue(); for (Mutation m : mutations) { if (m == null) { continue; } if (m instanceof Delete) { Put put = new Put(m.getRow()); put.addColumn(emptyCF, emptyCQ, IndexRegionObserver.getMaxTimestamp(m), IndexRegionObserver.UNVERIFIED_BYTES); addToMap(unverifiedIndexMutations, tableInfo, put); addToMap(verifiedOrDeletedIndexMutations, tableInfo, m); } else if (m instanceof Put) { long timestamp = IndexRegionObserver.getMaxTimestamp(m); if (m.has(emptyCF, emptyCQ)) { ((Put) m).addColumn(emptyCF, emptyCQ, timestamp, IndexRegionObserver.UNVERIFIED_BYTES); addToMap(unverifiedIndexMutations, tableInfo, m); } else { Put unverifiedPut = new Put((Put)m); unverifiedPut.addColumn(emptyCF, emptyCQ, timestamp, IndexRegionObserver.UNVERIFIED_BYTES); addToMap(verifiedOrDeletedIndexMutations, tableInfo, unverifiedPut); } Put verifiedPut = new Put(m.getRow()); verifiedPut.addColumn(emptyCF, emptyCQ, timestamp, IndexRegionObserver.VERIFIED_BYTES); addToMap(verifiedOrDeletedIndexMutations, tableInfo, verifiedPut); } else { addToMap(unverifiedIndexMutations, tableInfo, m); } } mapIter.remove(); } } }