[HBASE-10012] Hide ServerName constructor

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 0.98.0, 0.96.0
• Fix Version/s: 0.98.0, 0.96.1
• Component/s: Client
• Hadoop Flags: Reviewed
• Assignee: Nicolas Liochon
• Reporter: Nicolas Liochon
• Created: 20/Nov/13 11:57
• Updated: 10/Sep/14 19:17
• Resolved: 20/Nov/13 20:26

Description

If we want to share the instances of ServerName, we need to hide the constructor, and do as we do in TableName.

This patch is just about the refactoring of "new ServerName" to ServerName.valueOf, to make its review simpler.

Issue Links

• breaks: HBASE-10186 region_mover.rb broken because ServerName constructor is changed to private
• breaks: HBASE-10188 Deprecate ServerName constructors, but make it public.