This is an automated email from the ASF dual-hosted git repository.

janh pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hbase.git


The following commit(s) were added to refs/heads/master by this push:
     new 3dfa98f  HBASE-22258 Removed deprecated methods in VisibilityClient
3dfa98f is described below

commit 3dfa98ffc1f624f04936c04540cdb447b994df53
Author: Jan Hentschel <[email protected]>
AuthorDate: Wed Apr 17 13:14:50 2019 +0200

    HBASE-22258 Removed deprecated methods in VisibilityClient
---
 .../security/visibility/VisibilityClient.java      | 100 ---------------------
 1 file changed, 100 deletions(-)

diff --git 
a/hbase-client/src/main/java/org/apache/hadoop/hbase/security/visibility/VisibilityClient.java
 
b/hbase-client/src/main/java/org/apache/hadoop/hbase/security/visibility/VisibilityClient.java
index 35564d6..c780a80 100644
--- 
a/hbase-client/src/main/java/org/apache/hadoop/hbase/security/visibility/VisibilityClient.java
+++ 
b/hbase-client/src/main/java/org/apache/hadoop/hbase/security/visibility/VisibilityClient.java
@@ -26,10 +26,8 @@ import java.io.IOException;
 import java.util.Map;
 import java.util.regex.Pattern;
 
-import org.apache.hadoop.conf.Configuration;
 import org.apache.hadoop.hbase.HConstants;
 import org.apache.hadoop.hbase.client.Connection;
-import org.apache.hadoop.hbase.client.ConnectionFactory;
 import org.apache.hadoop.hbase.client.Table;
 import org.apache.hadoop.hbase.client.coprocessor.Batch;
 import org.apache.hadoop.hbase.client.security.SecurityCapability;
@@ -69,23 +67,6 @@ public class VisibilityClient {
   /**
    * Utility method for adding label to the system.
    *
-   * @param conf
-   * @param label
-   * @return VisibilityLabelsResponse
-   * @throws Throwable
-   * @deprecated Use {@link #addLabel(Connection,String)} instead.
-   */
-  @Deprecated
-  public static VisibilityLabelsResponse addLabel(Configuration conf, final 
String label)
-      throws Throwable {
-    try (Connection connection = ConnectionFactory.createConnection(conf)) {
-      return addLabels(connection, new String[] { label });
-    }
-  }
-
-  /**
-   * Utility method for adding label to the system.
-   *
    * @param connection
    * @param label
    * @return VisibilityLabelsResponse
@@ -99,23 +80,6 @@ public class VisibilityClient {
   /**
    * Utility method for adding labels to the system.
    *
-   * @param conf
-   * @param labels
-   * @return VisibilityLabelsResponse
-   * @throws Throwable
-   * @deprecated Use {@link #addLabels(Connection,String[])} instead.
-   */
-  @Deprecated
-  public static VisibilityLabelsResponse addLabels(Configuration conf, final 
String[] labels)
-      throws Throwable {
-    try (Connection connection = ConnectionFactory.createConnection(conf)) {
-      return addLabels(connection, labels);
-    }
-  }
-
-  /**
-   * Utility method for adding labels to the system.
-   *
    * @param connection
    * @param labels
    * @return VisibilityLabelsResponse
@@ -158,23 +122,6 @@ public class VisibilityClient {
 
   /**
    * Sets given labels globally authorized for the user.
-   * @param conf
-   * @param auths
-   * @param user
-   * @return VisibilityLabelsResponse
-   * @throws Throwable
-   * @deprecated Use {@link #setAuths(Connection,String[],String)} instead.
-   */
-  @Deprecated
-  public static VisibilityLabelsResponse setAuths(Configuration conf, final 
String[] auths,
-      final String user) throws Throwable {
-    try (Connection connection = ConnectionFactory.createConnection(conf)) {
-      return setOrClearAuths(connection, auths, user, true);
-    }
-  }
-
-  /**
-   * Sets given labels globally authorized for the user.
    * @param connection
    * @param auths
    * @param user
@@ -187,20 +134,6 @@ public class VisibilityClient {
   }
 
   /**
-   * @param conf
-   * @param user
-   * @return labels, the given user is globally authorized for.
-   * @throws Throwable
-   * @deprecated Use {@link #getAuths(Connection,String)} instead.
-   */
-  @Deprecated
-  public static GetAuthsResponse getAuths(Configuration conf, final String 
user) throws Throwable {
-    try (Connection connection = ConnectionFactory.createConnection(conf)) {
-      return getAuths(connection, user);
-    }
-  }
-
-  /**
    * @param connection the Connection instance to use.
    * @param user
    * @return labels, the given user is globally authorized for.
@@ -237,22 +170,6 @@ public class VisibilityClient {
 
   /**
    * Retrieve the list of visibility labels defined in the system.
-   * @param conf
-   * @param regex  The regular expression to filter which labels are returned.
-   * @return labels The list of visibility labels defined in the system.
-   * @throws Throwable
-   * @deprecated Use {@link #listLabels(Connection,String)} instead.
-   */
-  @Deprecated
-  public static ListLabelsResponse listLabels(Configuration conf, final String 
regex)
-      throws Throwable {
-    try(Connection connection = ConnectionFactory.createConnection(conf)){
-      return listLabels(connection, regex);
-    }
-  }
-
-  /**
-   * Retrieve the list of visibility labels defined in the system.
    * @param connection The Connection instance to use.
    * @param regex  The regular expression to filter which labels are returned.
    * @return labels The list of visibility labels defined in the system.
@@ -293,23 +210,6 @@ public class VisibilityClient {
 
   /**
    * Removes given labels from user's globally authorized list of labels.
-   * @param conf
-   * @param auths
-   * @param user
-   * @return VisibilityLabelsResponse
-   * @throws Throwable
-   * @deprecated Use {@link #clearAuths(Connection,String[],String)} instead.
-   */
-  @Deprecated
-  public static VisibilityLabelsResponse clearAuths(Configuration conf, final 
String[] auths,
-      final String user) throws Throwable {
-    try (Connection connection = ConnectionFactory.createConnection(conf)) {
-      return setOrClearAuths(connection, auths, user, false);
-    }
-  }
-
-  /**
-   * Removes given labels from user's globally authorized list of labels.
    * @param connection
    * @param auths
    * @param user

Reply via email to