From 50fd775bc4867aee9b6a86abefdb9d1661c1b979 Mon Sep 17 00:00:00 2001 From: pingsutw Date: Tue, 26 Mar 2019 18:22:15 +0800 Subject: [PATCH] HBASE-22102 Remove AsyncAdmin.isTableAvailable(TableName, byte[][]) --- .../main/java/org/apache/hadoop/hbase/client/AsyncAdmin.java | 12 ------------ .../java/org/apache/hadoop/hbase/client/AsyncHBaseAdmin.java | 5 ----- .../org/apache/hadoop/hbase/client/RawAsyncHBaseAdmin.java | 7 ------- .../apache/hadoop/hbase/client/TestAsyncTableAdminApi2.java | 10 ---------- 4 files changed, 34 deletions(-) diff --git a/hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncAdmin.java b/hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncAdmin.java index f7adc16510..c9a57d38fe 100644 --- a/hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncAdmin.java +++ b/hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncAdmin.java @@ -220,18 +220,6 @@ public interface AsyncAdmin { */ CompletableFuture isTableAvailable(TableName tableName); - /** - * Use this api to check if the table has been created with the specified number of splitkeys - * which was used while creating the given table. Note : If this api is used after a table's - * region gets splitted, the api may return false. The return value will be wrapped by a - * {@link CompletableFuture}. - * @param tableName name of table to check - * @param splitKeys keys to check if the table has been created with all split keys - * @deprecated Since 2.2.0. Will be removed in 3.0.0. Use {@link #isTableAvailable(TableName)} - */ - @Deprecated - CompletableFuture isTableAvailable(TableName tableName, byte[][] splitKeys); - /** * Add a column family to an existing table. * @param tableName name of the table to add column family to diff --git a/hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncHBaseAdmin.java b/hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncHBaseAdmin.java index 0eceaad4ce..f1b076030a 100644 --- a/hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncHBaseAdmin.java +++ b/hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncHBaseAdmin.java @@ -175,11 +175,6 @@ class AsyncHBaseAdmin implements AsyncAdmin { return wrap(rawAdmin.isTableAvailable(tableName)); } - @Override - public CompletableFuture isTableAvailable(TableName tableName, byte[][] splitKeys) { - return wrap(rawAdmin.isTableAvailable(tableName, splitKeys)); - } - @Override public CompletableFuture addColumnFamily(TableName tableName, ColumnFamilyDescriptor columnFamily) { diff --git a/hbase-client/src/main/java/org/apache/hadoop/hbase/client/RawAsyncHBaseAdmin.java b/hbase-client/src/main/java/org/apache/hadoop/hbase/client/RawAsyncHBaseAdmin.java index 587c6e2135..7d130a8484 100644 --- a/hbase-client/src/main/java/org/apache/hadoop/hbase/client/RawAsyncHBaseAdmin.java +++ b/hbase-client/src/main/java/org/apache/hadoop/hbase/client/RawAsyncHBaseAdmin.java @@ -684,13 +684,6 @@ class RawAsyncHBaseAdmin implements AsyncAdmin { return isTableAvailable(tableName, Optional.empty()); } - @Override - public CompletableFuture isTableAvailable(TableName tableName, byte[][] splitKeys) { - Preconditions.checkNotNull(splitKeys, "splitKeys is null. If you don't specify splitKeys," - + " use isTableAvailable(TableName) instead"); - return isTableAvailable(tableName, Optional.of(splitKeys)); - } - private CompletableFuture isTableAvailable(TableName tableName, Optional splitKeys) { if (TableName.isMetaTableName(tableName)) { diff --git a/hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestAsyncTableAdminApi2.java b/hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestAsyncTableAdminApi2.java index 63ff3e48ad..4f5cbf219d 100644 --- a/hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestAsyncTableAdminApi2.java +++ b/hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestAsyncTableAdminApi2.java @@ -209,16 +209,6 @@ public class TestAsyncTableAdminApi2 extends TestAsyncAdminBase { } } - - @Test - public void testTableAvailableWithRandomSplitKeys() throws Exception { - createTableWithDefaultConf(tableName); - byte[][] splitKeys = new byte[1][]; - splitKeys = new byte[][] { new byte[] { 1, 1, 1 }, new byte[] { 2, 2, 2 } }; - boolean tableAvailable = admin.isTableAvailable(tableName, splitKeys).get(); - assertFalse("Table should be created with 1 row in META", tableAvailable); - } - @Test public void testCompactionTimestamps() throws Exception { createTableWithDefaultConf(tableName); -- 2.16.2.windows.1