[HBASE-14954] IllegalArgumentException was thrown when doing online configuration change in CompactSplitThread

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.1.2
• Fix Version/s: 1.2.0, 1.3.0, 1.1.3, 2.0.0
• Component/s: Compaction, regionserver
• Hadoop Flags: Reviewed
• Assignee: Victor Xu
• Reporter: Victor Xu
• Created: 09/Dec/15 02:21
• Updated: 27/Jan/16 15:28
• Resolved: 09/Dec/15 15:22

Description

Online configuration change is a terrific feature for HBase administrators. However, when we use this feature to tune compaction thread pool size online, it triggered a IllegalArgumentException. The cause is the order of setMaximumPoolSize() and setCorePoolSize() of ThreadPoolExecutor: when turning parameters bigger, we should setMax first; when turning parameters smaller, we need to setCore first. Besides, there is also a copy-code bug in merge and split thread pool which I will fix together.