You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by jm...@apache.org on 2013/12/17 20:28:59 UTC

svn commit: r1551662 - /hbase/tags/0.96.1/hbase-client/src/main/java/org/apache/hadoop/hbase/ServerName.java

Author: jmhsieh
Date: Tue Dec 17 19:28:59 2013
New Revision: 1551662

URL: http://svn.apache.org/r1551662
Log:
HBASE-10188 Deprecate ServerName constructors, but make it public (Nicolas Liochon)

Modified:
    hbase/tags/0.96.1/hbase-client/src/main/java/org/apache/hadoop/hbase/ServerName.java

Modified: hbase/tags/0.96.1/hbase-client/src/main/java/org/apache/hadoop/hbase/ServerName.java
URL: http://svn.apache.org/viewvc/hbase/tags/0.96.1/hbase-client/src/main/java/org/apache/hadoop/hbase/ServerName.java?rev=1551662&r1=1551661&r2=1551662&view=diff
==============================================================================
--- hbase/tags/0.96.1/hbase-client/src/main/java/org/apache/hadoop/hbase/ServerName.java (original)
+++ hbase/tags/0.96.1/hbase-client/src/main/java/org/apache/hadoop/hbase/ServerName.java Tue Dec 17 19:28:59 2013
@@ -94,7 +94,11 @@ public class ServerName implements Compa
   private byte [] bytes;
   public static final List<ServerName> EMPTY_SERVER_LIST = new ArrayList<ServerName>(0);
 
-  private ServerName(final String hostname, final int port, final long startcode) {
+  /**
+   * @deprecated since 0.96, made private in 0.98. use {@link #valueOf(String, int, long)} instead.
+   */
+  @Deprecated
+  public ServerName(final String hostname, final int port, final long startcode) {
     // Drop the domain is there is one; no need of it in a local cluster.  With it, we get long
     // unwieldy names.
     this.hostnameOnly = hostname;
@@ -114,12 +118,20 @@ public class ServerName implements Compa
     return parts[0];
   }
 
-  private ServerName(final String serverName) {
+  /**
+   * @deprecated since 0.96, made private in 0.98. use {@link #valueOf(String)} instead.
+   */
+  @Deprecated
+  public ServerName(final String serverName) {
     this(parseHostname(serverName), parsePort(serverName),
       parseStartcode(serverName));
   }
 
-  private ServerName(final String hostAndPort, final long startCode) {
+  /**
+   * @deprecated since 0.96, made private in 0.98+. use {@link #valueOf(String, long)} instead.
+   */
+  @Deprecated
+  public ServerName(final String hostAndPort, final long startCode) {
     this(Addressing.parseHostname(hostAndPort),
       Addressing.parsePort(hostAndPort), startCode);
   }