Re: svn commit: r1099971 - /hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/HServerAddress.java

[Gaojinchao]

Hi stack:
Do it also need apply 3744-addendum-for-TestAdmin.patch by Ted Yu? 

https://issues.apache.org/jira/browse/HBASE-3744

-----邮件原件-----
发件人: st...@apache.org [mailto:st...@apache.org] 
发送时间: 2011年5月6日 5:20
收件人: comm...@hbase.apache.org
主题: svn commit: r1099971 - 
/hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/HServerAddress.java

Author: stack
Date: Thu May  5 21:20:08 2011
New Revision: 1099971

URL: http://svn.apache.org/viewvc?rev=1099971&view=rev
Log:
Fix up jenkins build failure in TestAdmin.testCreateTableWithRegions... address 
can be null when doing compareTo

Modified:
    
hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/HServerAddress.java

Modified: 
hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/HServerAddress.java
URL: 
http://svn.apache.org/viewvc/hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/HServerAddress.java?rev=1099971&r1=1099970&r2=1099971&view=diff
==============================================================================
--- 
hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/HServerAddress.java 
(original)
+++ 
hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/HServerAddress.java 
Thu May  5 21:20:08 2011
@@ -187,6 +187,8 @@ public class HServerAddress implements W
     // Addresses as Strings may not compare though address is for the one
     // server with only difference being that one address has hostname
     // resolved whereas other only has IP.
+    if (this.address == null) return -1;
+    if (o.address == null) return 1;
     if (address.equals(o.address)) return 0;
     return toString().compareTo(o.toString());
   }