[HBASE-17030] Procedure v2 - A couple of tweaks to the SplitTableRegionProcedure

• Type: Sub-task
• Status: Closed
• Priority: Trivial
• Resolution: Fixed
• Affects Version/s: 2.0.0
• Fix Version/s: 2.0.0
• Component/s: proc-v2
• Assignee: Matteo Bertozzi
• Reporter: Matteo Bertozzi
• Created: 04/Nov/16 23:30
• Updated: 17/Jun/22 18:09
• Resolved: 08/Nov/16 18:42

Description

Make a couple of tweaks to HBASE-14551 split procedure

• remove tableName from SplitTableRegionProcedure ctor since we have the RegionInfo that contains the name already
• move the checkRow in the constructor of the SplitTableRegionProcedure, since the splitRow will never change and we can avoid to start the proc if we have a bad splitRow.
• use the base AbstractStateMachineTableProcedure for the "user" field
• remove protobuf fields that can be extrapolated from other info (table_name, split_row)
• avoid htd lookup every family iteration of splitStoreFiles()

Issue Links

• links to: Reviewboard