[HBASE-17174] Refactor the AsyncProcess, BufferedMutatorImpl, and HTable

• Type: New Feature
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 2.0.0
• Fix Version/s: 2.0.0
• Hadoop Flags: Reviewed

Release Note

+ cleanup some unused code
+ allow being able to share pool between BufferedMutatorImpl
+ setting "hbase.client.request.controller.impl" to the name of the alternate RequestController (traffic control) implementation class in Configuration
+ The default RequestController implementation is SimpleRequestController
+ setting "hbase.client.log.detail.period.ms" to call logger on a period when waiting for tasks to complete

• Assignee: Chia-Ping Tsai
• Reporter: Chia-Ping Tsai
• Created: 24/Nov/16 11:21
• Updated: 27/Dec/16 11:32
• Resolved: 24/Dec/16 04:12

Description

The following are reasons of refactoring.

1. A update-heavy application, for example, loader, creates many BufferedMutator for batch updates. But these BufferedMutators can’t share a large threadpool because the shutdown() method will be called when closing any BufferedMutator. This patch adds a flag into BufferedMutatorParams for preventing calling the shutdown() method in BufferedMutatorImpl#close
2. The AsyncProcess has the powerful traffic control, but the control is against the single table currently. We should allow alternate traffic control implementation for advanced user who want more control.

All suggestions are welcome.

Issue Links

• links to: reviewboard