[HBASE-22108] Avoid passing null in Admin methods

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

Release Note

Introduced these methods:
void move(byte[]);
void move(byte[], ServerName);
Future<Void> splitRegionAsync(byte[]);

These methods are deprecated:
void move(byte[], byte[])

• Assignee: Duo Zhang
• Reporter: Duo Zhang
• Created: 26/Mar/19 02:25
• Updated: 07/Apr/19 23:38
• Resolved: 07/Apr/19 13:18

Description

For some methods we must pass null if we want specific behaviors, for example, move region to a random server, split region without providing an explicit split point, etc.

We should provide methods without some parameters so user do not need to pass null.

So in the future users do not need to guess whether a method accept null parameters, the answer is always no.

Issue Links

• links to: Review Board