[HBASE-21750] Most of KeyValueUtil#length can be replaced by cell#getSerializedSize for better performance because the latter one has been optimized

• Type: Sub-task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 3.0.0-alpha-1, 2.2.0
• Hadoop Flags: Reviewed
• Assignee: Zheng Hu
• Reporter: Zheng Hu
• Created: 21/Jan/19 07:19
• Updated: 23/Jan/19 04:17
• Resolved: 22/Jan/19 12:35

Description

After HBASE-21657, Most subclass of Cell has a cached serialized size (Except those cells with tags), so I think most of the KeyValueUtil#length can be replaced by cell#getSerializedSize. Such as:

• KeyValueUtil.length in StoreFlusher#performFlush;
• KeyValueUtil.length in Compactor#performCompaction ;

and so on..

Will prepare a patch for this.

Issue Links

• links to: Review board