impl(v3): remove deprecated Bigtable Admin classes

Author: scotthart

doc/v3-migration-guide.md

@@ -487,6 +487,120 @@ internal legacy files.
 
 </details>
 
+<details>
+<summary>Removed <code>bigtable::AdminClient</code> and <code>bigtable::TableAdmin</code></summary>
+
+The `bigtable::AdminClient` class and `bigtable::TableAdmin` class have been
+replaced with `bigtable_admin::BigtableTableAdminClient`.
+
+**Before:**
+
+```cpp
+
+std::shared_ptr<bigtable::AdminClient> admin_client = 
+    bigtable::MakeAdminClient("project-id");
+auto table_admin = std::make_unique<bigtable::TableAdmin>(
+    admin_client, "instance-id");
+
+// Drop a selection of rows by key prefix.
+auto result = table_admin.DropRowByPrefix("table-id", "row-key-prefix");
+
+// Drop all rows.
+result = table_admin.DropAllRows("table-id");
+```
+
+**After:**
+
+```cpp
+#include "google/cloud/bigtable/admin/bigtable_table_admin_client.h"
+
+auto table_admin = bigtable_admin::BigtableTableAdminClient(
+    bigtable_admin::MakeBigtableAdminConnection());
+auto table_name = bigtable::TableName("project-id", "instance-id", "table-id");
+
+// Drop a selection of rows by key prefix.
+google::bigtable::admin::v2::DropRowRangeRequest drop_rows_by_prefix;
+drop_rows_by_prefix.set_name(table_name);
+drop_rows_by_prefix.set_row_key_prefix("row-key-prefix");
+auto result = table_admin.DropRowRange(drop_rows_by_prefix);
+
+// Drop all rows.
+google::bigtable::admin::v2::DropRowRangeRequest drop_all_rows;
+drop_all_rows.set_name(table_name);
+drop_all_rows.set_delete_all_data_from_table(true);
+result = table_admin.DropRowRange(drop_all_rows);
+```
+
+</details>
+
+<details><summary><code>WaitForConsistency</code> is now a free function</summary>
+
+With the removal of the `bigtable::TableAdmin` class, `WaitForConsistency` is
+now a free function.
+
+**Before:**
+
+```cpp
+
+std::shared_ptr<bigtable::AdminClient> admin_client = 
+    bigtable::MakeAdminClient("project-id");
+auto table_admin = std::make_unique<bigtable::TableAdmin>(
+    admin_client, "instance-id");
+
+auto token = table_admin.GenerateConsistencyToken("table-id");
+if (!token) throw std::runtime_error(token.status().message());
+auto result = table_admin.WaitForConsistency("table-id", *token);
+```
+
+**After:**
+
+```cpp
+#include "google/cloud/bigtable/admin/bigtable_table_admin_client.h"
+#include "google/cloud/bigtable/wait_for_consistency.h"
+
+auto connection = bigtable_admin::MakeBigtableAdminConnection();
+auto table_admin = bigtable_admin::BigtableTableAdminClient(connection);
+auto table_name = bigtable::TableName("project-id", "instance-id", "table-id");
+
+auto token = table_admin.GenerateConsistencyToken(table_name);
+if (!token) throw std::runtime_error(token.status().message());
+auto result = bigtable_admin::WaitForConsistency(connection, table_name,
+                                                 token->consistency_token());
+```
+
+</details>
+
+<details>
+<summary>Removed <code>bigtable::InstanceAdminClient</code> and <code>bigtable::InstanceAdmin</code></summary>
+
+The `bigtable::InstanceAdminClient` class and `bigtable::InstanceAdmin` class
+have been replaced with `bigtable_admin::BigtableInstanceAdminClient`.
+
+**Before:**
+
+```cpp
+auto instance_admin_client = bigtable::MakeInstanceAdminClient("project-id");
+auto instance_admin =
+    std::make_unique<bigtable::InstanceAdmin>(instance_admin_client);
+
+auto clusters = instance_admin->ListClusters();
+```
+
+**After:**
+
+```cpp
+#include "google/cloud/bigtable/admin/bigtable_instance_admin_client.h"
+
+auto instance_admin =
+        std::make_unique<bigtable_admin::BigtableInstanceAdminClient>(
+            bigtable_admin::MakeBigtableInstanceAdminConnection());
+
+auto clusters = instance_admin->ListClusters(
+    InstanceName("project-id", "instance-id"));
+```
+
+</details>
+
 ### Pubsub
 
 <details>

google/cloud/bigtable/CMakeLists.txt

@@ -104,8 +104,6 @@ add_library(
     admin/internal/bigtable_table_admin_tracing_connection.h
     admin/internal/bigtable_table_admin_tracing_stub.cc
     admin/internal/bigtable_table_admin_tracing_stub.h
-    admin_client.cc
-    admin_client.h
     app_profile_config.cc
     app_profile_config.h
     bound_query.cc
@@ -131,10 +129,6 @@ add_library(
     iam_policy.h
     idempotent_mutation_policy.cc
     idempotent_mutation_policy.h
-    instance_admin.cc
-    instance_admin.h
-    instance_admin_client.cc
-    instance_admin_client.h
     instance_config.cc
     instance_config.h
     instance_list_responses.h
@@ -259,8 +253,6 @@ add_library(
     sql_statement.h
     table.cc
     table.h
-    table_admin.cc
-    table_admin.h
     table_config.cc
     table_config.h
     table_resource.cc
@@ -424,7 +416,6 @@ if (BUILD_TESTING)
     # List the unit tests, then setup the targets and dependencies.
     set(bigtable_client_unit_tests
         # cmake-format: sort
-        admin_client_test.cc
         app_profile_config_test.cc
         async_read_stream_test.cc
         bigtable_version_test.cc
@@ -441,8 +432,6 @@ if (BUILD_TESTING)
         iam_binding_test.cc
         iam_policy_test.cc
         idempotent_mutation_policy_test.cc
-        instance_admin_client_test.cc
-        instance_admin_test.cc
         instance_config_test.cc
         instance_resource_test.cc
         instance_update_config_test.cc
@@ -490,7 +479,6 @@ if (BUILD_TESTING)
         rpc_backoff_policy_test.cc
         rpc_retry_policy_test.cc
         sql_statement_test.cc
-        table_admin_test.cc
         table_config_test.cc
         table_resource_test.cc
         table_test.cc

google/cloud/bigtable/admin_client.cc

@@ -17,7 +17,6 @@
 """Automatically generated unit tests list - DO NOT EDIT."""
 
 bigtable_client_unit_tests = [
-    "admin_client_test.cc",
     "app_profile_config_test.cc",
     "async_read_stream_test.cc",
     "bigtable_version_test.cc",
@@ -34,8 +33,6 @@ bigtable_client_unit_tests = [
     "iam_binding_test.cc",
     "iam_policy_test.cc",
     "idempotent_mutation_policy_test.cc",
-    "instance_admin_client_test.cc",
-    "instance_admin_test.cc",
     "instance_config_test.cc",
     "instance_resource_test.cc",
     "instance_update_config_test.cc",
@@ -83,7 +80,6 @@ bigtable_client_unit_tests = [
     "rpc_backoff_policy_test.cc",
     "rpc_retry_policy_test.cc",
     "sql_statement_test.cc",
-    "table_admin_test.cc",
     "table_config_test.cc",
     "table_resource_test.cc",
     "table_test.cc",

google/cloud/bigtable/admin_client.h

@@ -59,8 +59,8 @@ source module /google/cloud/bigtable/tools/run_emulator_utils.sh
 production_only_targets=(
   "//google/cloud/bigtable/examples:bigtable_table_admin_backup_snippets"
   "//google/cloud/bigtable/examples:table_admin_iam_policy_snippets"
-  "//google/cloud/bigtable/tests:admin_backup_integration_test"
-  "//google/cloud/bigtable/tests:admin_iam_policy_integration_test"
+  "//google/cloud/bigtable/tests:table_admin_backup_integration_test"
+  "//google/cloud/bigtable/tests:table_admin_iam_policy_integration_test"
 )
 
 # Coverage builds are more subject to flakiness, as we must explicitly disable