[HBASE-16225] Refactor ScanQueryMatcher

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.4.0, 2.0.0
• Fix Version/s: 1.4.0, 2.0.0
• Component/s: regionserver, Scanners
• Hadoop Flags: Reviewed
• Assignee: Duo Zhang
• Reporter: Duo Zhang
• Created: 13/Jul/16 13:51
• Updated: 20/Sep/18 18:38
• Resolved: 02/Aug/16 06:09

Description

As said in HBASE-16223, the code of ScanQueryMatcher is too complicated. I suggest that we can abstract an interface and implement several sub classes which separate different logic into different implementations. For example, the requirements of compaction and user scan are different, now we also need to consider the logic of user scan even if we only want to add a logic for compaction. And at least, the raw scan does not need a query matcher... we can implement a dummy query matcher for it.

Suggestions are welcomed. Thanks.

Issue Links

• blocks: HBASE-16223 Drop duplicated delete markers in minor compaction
• is related to: HBASE-3562 ValueFilter is being evaluated before performing the column match
• is related to: HBASE-9440 Pass blocks of KVs from HFile scanner to the StoreFileScanner and up
• is related to: HBASE-11084 Handle deletions and column tracking using internal filters rather than using DeleteTrackers and ColumnTrackers
• links to: reviewboard