[HBASE-16931] Setting cell's seqId to zero in compaction flow might cause RS down.

• Type: Bug
• Status: Closed
• Priority: Critical
• Resolution: Fixed
• Affects Version/s: 1.3.0, 1.4.0, 1.2.3, 1.1.7, 2.0.0
• Fix Version/s: 1.3.0, 1.2.4, 1.1.8, 2.0.0
• Component/s: regionserver
• Hadoop Flags: Reviewed
• Assignee: Lijin Bin
• Reporter: Lijin Bin
• Created: 24/Oct/16 04:05
• Updated: 09/Nov/17 02:47
• Resolved: 02/Nov/16 03:41

Description

Compactor#performCompaction
do {
hasMore = scanner.next(cells, scannerContext);
// output to writer:
for (Cell c : cells) {
if (cleanSeqId && c.getSequenceId() <= smallestReadPoint)

{ CellUtil.setSequenceId(c, 0); }

writer.append(c);
}
cells.clear();
} while (hasMore);
scanner.next will choose at most "hbase.hstore.compaction.kv.max" kvs, the last cell still reference by StoreScanner.prevCell, so if cleanSeqId is called when the scanner.next call StoreScanner.checkScanOrder may throw exception and cause regionserver down.