[HBASE-20711] Save on a Cell iteration when writing

• Type: Bug
• Status: Patch Available
• Priority: Minor
• Resolution: Unresolved
• Component/s: Performance
• Assignee: Michael Stack
• Reporter: Michael Stack
• Created: 09/Jun/18 15:23
• Updated: 22/Jan/20 13:48

Description

This is a minor savings. We were doing a spin through all Cells on receipt just to check their size when subsequently, we were doing an iteration of all Cells to insert. It manifest as a little spike in perf output. This change removes the purposed spin through Cells and just does size check as part of the general Cell insert (perf spike no longer shows but the cost of the size check still remains).

There is also a minor bug fix where on receipt we were using the Puts row rather than the Cells row; client may have succeeded in submitting a Cell that disagreed with the hosting Mutation and it would have been written as something else altogether – with the Puts row – rather than being rejected.

Issue Links

• links to: Review Board (branch-2.0)