[ 
https://issues.apache.org/jira/browse/HBASE-18235?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16054579#comment-16054579
 ] 

Umesh Agashe edited comment on HBASE-18235 at 6/19/17 6:54 PM:
---------------------------------------------------------------

{code}
   // Used to signal to the caller that the region(s) cannot be assigned
-  static final ServerName BOGUS_SERVER_NAME = 
ServerName.valueOf("bogus.example.com,1,1");
+  ServerName BOGUS_SERVER_NAME = ServerName.valueOf("bogus.example.com,1,1");
{code}

I did a cleanup while working on HBASE-17786. As the field is in interface 
static final is redundant. Removed 'static final' from the declaration. Change 
'localhost' to 'bogus.example.com' happened with patch for HBASE-17624, as 
[~toffer] has pointed out.


was (Author: uagashe):
{code}
   // Used to signal to the caller that the region(s) cannot be assigned
-  static final ServerName BOGUS_SERVER_NAME = 
ServerName.valueOf("bogus.example.com,1,1");
+  ServerName BOGUS_SERVER_NAME = ServerName.valueOf("bogus.example.com,1,1");
{code}

I did a cleanup while working on HBASE-17786. As the field is in interface 
static is redundant. Removed static from the declaration. Change 'localhost' to 
'bogus.example.com' happened with patch for HBASE-17624, as [~toffer] has 
pointed out.

> LoadBalancer.BOGUS_SERVER_NAME should not have a bogus hostname
> ---------------------------------------------------------------
>
>                 Key: HBASE-18235
>                 URL: https://issues.apache.org/jira/browse/HBASE-18235
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Francis Liu
>            Assignee: Francis Liu
>
> The original patch used localhost to have assignment fail fast. Avoiding 
> misleading DNS exceptions, delays due to dns lookup, etc. 
> Was wondering what the reason was for changing it?



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to