feat: remove DataClient and it's usages

Author: mpeddada1

examples/gcs2cbt.cc

@@ -60,8 +60,9 @@ int main(int argc, char* argv[]) try {
 
   // Create a connection to Cloud Bigtable and an object to manipulate the
   // specific table used in this demo.
-  cbt::Table table(cbt::MakeDataClient(options.project_id, options.instance_id),
-                   options.table_id);
+  cbt::Table table(cbt::MakeDataConnection(),
+                   cbt::TableResource(options.project_id, options.instance_id,
+                                      options.table_id));
   cbt::MutationBatcher batcher(table);
 
   // How often do we print a progress marker ('.') in the reader thread.

google/cloud/bigtable/CMakeLists.txt

@@ -123,8 +123,6 @@ add_library(
     cluster_list_responses.h
     column_family.h
     completion_queue.h
-    data_client.cc
-    data_client.h
     data_connection.cc
     data_connection.h
     expr.cc
@@ -403,8 +401,6 @@ if (BUILD_TESTING)
         testing/cleanup_stale_resources.h
         testing/embedded_server_test_fixture.cc
         testing/embedded_server_test_fixture.h
-        testing/inprocess_data_client.cc
-        testing/inprocess_data_client.h
         testing/mock_async_failing_rpc_factory.h
         testing/mock_bigtable_stub.h
         testing/mock_data_client.h
@@ -456,7 +452,6 @@ if (BUILD_TESTING)
         client_test.cc
         cluster_config_test.cc
         column_family_test.cc
-        data_client_test.cc
         data_connection_test.cc
         expr_test.cc
         filters_test.cc
@@ -524,13 +519,10 @@ if (BUILD_TESTING)
         table_admin_test.cc
         table_apply_test.cc
         table_bulk_apply_test.cc
-        table_check_and_mutate_row_test.cc
         table_config_test.cc
-        table_readmodifywriterow_test.cc
         table_readrow_test.cc
         table_readrows_test.cc
         table_resource_test.cc
-        table_sample_row_keys_test.cc
         table_test.cc
         testing/cleanup_stale_resources_test.cc
         testing/random_names_test.cc

google/cloud/bigtable/benchmarks/benchmark.cc

@@ -16,6 +16,7 @@
 #include "google/cloud/bigtable/admin/bigtable_table_admin_client.h"
 #include "google/cloud/bigtable/benchmarks/random_mutation.h"
 #include "google/cloud/bigtable/resource_names.h"
+#include "google/cloud/grpc_options.h"
 #include "google/cloud/internal/background_threads_impl.h"
 #include "google/cloud/internal/getenv.h"
 #include "google/cloud/internal/make_status.h"

google/cloud/bigtable/benchmarks/embedded_server_test.cc

@@ -16,6 +16,7 @@
 #include "google/cloud/bigtable/admin/bigtable_table_admin_client.h"
 #include "google/cloud/bigtable/resource_names.h"
 #include "google/cloud/bigtable/table.h"
+#include "google/cloud/grpc_options.h"
 #include "google/cloud/testing_util/status_matchers.h"
 #include <gmock/gmock.h>
 #include <thread>

google/cloud/bigtable/bigtable_client_testing.bzl

@@ -19,10 +19,8 @@
 bigtable_client_testing_hdrs = [
     "testing/cleanup_stale_resources.h",
     "testing/embedded_server_test_fixture.h",
-    "testing/inprocess_data_client.h",
     "testing/mock_async_failing_rpc_factory.h",
     "testing/mock_bigtable_stub.h",
-    "testing/mock_data_client.h",
     "testing/mock_mutate_rows_limiter.h",
     "testing/mock_mutate_rows_reader.h",
     "testing/mock_partial_result_set_reader.h",
@@ -32,14 +30,11 @@ bigtable_client_testing_hdrs = [
     "testing/mock_sample_row_keys_reader.h",
     "testing/random_names.h",
     "testing/table_integration_test.h",
-    "testing/table_test_fixture.h",
 ]
 
 bigtable_client_testing_srcs = [
     "testing/cleanup_stale_resources.cc",
     "testing/embedded_server_test_fixture.cc",
-    "testing/inprocess_data_client.cc",
     "testing/random_names.cc",
     "testing/table_integration_test.cc",
-    "testing/table_test_fixture.cc",
 ]

google/cloud/bigtable/bigtable_client_unit_tests.bzl

@@ -28,7 +28,6 @@ bigtable_client_unit_tests = [
     "client_test.cc",
     "cluster_config_test.cc",
     "column_family_test.cc",
-    "data_client_test.cc",
     "data_connection_test.cc",
     "expr_test.cc",
     "filters_test.cc",
@@ -57,12 +56,6 @@ bigtable_client_unit_tests = [
     "internal/default_row_reader_test.cc",
     "internal/defaults_test.cc",
     "internal/google_bytes_traits_test.cc",
-    "internal/legacy_async_bulk_apply_test.cc",
-    "internal/legacy_async_row_reader_test.cc",
-    "internal/legacy_async_row_sampler_test.cc",
-    "internal/legacy_bulk_mutator_test.cc",
-    "internal/legacy_row_reader_test.cc",
-    "internal/logging_data_client_test.cc",
     "internal/logging_result_set_reader_test.cc",
     "internal/metrics_test.cc",
     "internal/mutate_rows_limiter_test.cc",
@@ -76,7 +69,6 @@ bigtable_client_unit_tests = [
     "internal/retry_traits_test.cc",
     "internal/traced_row_reader_test.cc",
     "internal/tuple_utils_test.cc",
-    "legacy_table_test.cc",
     "metadata_update_policy_test.cc",
     "mocks/mock_row_reader_test.cc",
     "mutation_batcher_test.cc",
@@ -94,15 +86,8 @@ bigtable_client_unit_tests = [
     "rpc_retry_policy_test.cc",
     "sql_statement_test.cc",
     "table_admin_test.cc",
-    "table_apply_test.cc",
-    "table_bulk_apply_test.cc",
-    "table_check_and_mutate_row_test.cc",
     "table_config_test.cc",
-    "table_readmodifywriterow_test.cc",
-    "table_readrow_test.cc",
-    "table_readrows_test.cc",
     "table_resource_test.cc",
-    "table_sample_row_keys_test.cc",
     "table_test.cc",
     "testing/cleanup_stale_resources_test.cc",
     "testing/random_names_test.cc",