[HBASE-21782] LoadIncrementalHFiles should not be IA.Public

• Type: Task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 3.0.0-alpha-1, 2.2.0
• Component/s: mapreduce
• Labels: bulkload
• Hadoop Flags: Reviewed

Release Note

Introduce a BulkLoadHFiles interface which is marked as IA.Public, for doing bulk load programmatically.
Introduce a BulkLoadHFilesTool which extends BulkLoadHFiles, and is marked as IA.LimitedPrivate(TOOLS), for using from command line.
The old LoadIncrementalHFiles is deprecated and will be removed in 3.0.0.

• Assignee: Duo Zhang
• Reporter: Duo Zhang
• Created: 25/Jan/19 07:20
• Updated: 26/Jun/20 17:30
• Resolved: 27/Jan/19 03:38

Description

It is an implementation class, so some of the methods which are only supposed to be used by replication sink are also public to users. And it exposes methods which take Table and Connection as parameter and inside the implementation we assume that they are HTable and ConnectionImplementation, which will be a pain when we want to replace the sync client implementation with async client.

Here I think we should make the implementation class as IA.LimitPrivate(TOOL), and introduce an interface for bulking hfiles programmatically.

Issue Links

• relates to: HBASE-24644 Add a clause to the book noting that sometimes we short-circuit the deprecation cycle
• links to: Review Board