feat(bigtable): remove deprecated RowReader constructors

Author: mpeddada1

doc/v3-migration-guide.md

@@ -46,6 +46,18 @@ module which can be added to your `MODULE.bazel` file as a dependency.
 
 ### Bigtable
 
+<details>
+<summary>Bigtable: Removed <code>bigtable::RowReader</code> constructors
+</summary>
+<br>
+
+The `bigtable::RowReader` constructors that accept `DataClient` as
+an argument have been removed.
+
+Developers that read rows by directly constructing a `RowReader` object should instead construct a `Table` object and call `Table::ReadRows(...)`.
+
+</details>
+
 ### Pubsub
 
 ### Spanner

google/cloud/bigtable/row_reader.cc

@@ -65,32 +65,6 @@ RowReader::RowReader()
     : RowReader(
           std::make_shared<bigtable_internal::StatusOnlyRowReader>(Status{})) {}
 
-RowReader::RowReader(
-    std::shared_ptr<DataClient> client, std::string table_name, RowSet row_set,
-    std::int64_t rows_limit, Filter filter,
-    std::unique_ptr<RPCRetryPolicy> retry_policy,
-    std::unique_ptr<RPCBackoffPolicy> backoff_policy,
-    MetadataUpdatePolicy metadata_update_policy,
-    std::unique_ptr<internal::ReadRowsParserFactory> parser_factory)
-    : RowReader(std::make_shared<bigtable_internal::LegacyRowReader>(
-          std::move(client), std::move(table_name), std::move(row_set),
-          rows_limit, std::move(filter), std::move(retry_policy),
-          std::move(backoff_policy), std::move(metadata_update_policy),
-          std::move(parser_factory))) {}
-
-RowReader::RowReader(
-    std::shared_ptr<DataClient> client, std::string app_profile_id,
-    std::string table_name, RowSet row_set, std::int64_t rows_limit,
-    Filter filter, std::unique_ptr<RPCRetryPolicy> retry_policy,
-    std::unique_ptr<RPCBackoffPolicy> backoff_policy,
-    MetadataUpdatePolicy metadata_update_policy,
-    std::unique_ptr<internal::ReadRowsParserFactory> parser_factory)
-    : RowReader(std::make_shared<bigtable_internal::LegacyRowReader>(
-          std::move(client), std::move(app_profile_id), std::move(table_name),
-          std::move(row_set), rows_limit, std::move(filter),
-          std::move(retry_policy), std::move(backoff_policy),
-          std::move(metadata_update_policy), std::move(parser_factory))) {}
-
 std::int64_t constexpr RowReader::NO_ROWS_LIMIT;
 
 // The name must be all lowercase to work with range-for loops.

google/cloud/bigtable/row_reader.h

@@ -65,22 +65,6 @@ class RowReader {
   /// Default constructs an empty RowReader.
   RowReader();
 
-  GOOGLE_CLOUD_CPP_BIGTABLE_ROW_READER_CTOR_DEPRECATED()
-  RowReader(std::shared_ptr<DataClient> client, std::string table_name,
-            RowSet row_set, std::int64_t rows_limit, Filter filter,
-            std::unique_ptr<RPCRetryPolicy> retry_policy,
-            std::unique_ptr<RPCBackoffPolicy> backoff_policy,
-            MetadataUpdatePolicy metadata_update_policy,
-            std::unique_ptr<internal::ReadRowsParserFactory> parser_factory);
-
-  GOOGLE_CLOUD_CPP_BIGTABLE_ROW_READER_CTOR_DEPRECATED()
-  RowReader(std::shared_ptr<DataClient> client, std::string app_profile_id,
-            std::string table_name, RowSet row_set, std::int64_t rows_limit,
-            Filter filter, std::unique_ptr<RPCRetryPolicy> retry_policy,
-            std::unique_ptr<RPCBackoffPolicy> backoff_policy,
-            MetadataUpdatePolicy metadata_update_policy,
-            std::unique_ptr<internal::ReadRowsParserFactory> parser_factory);
-
   // NOLINTNEXTLINE(performance-noexcept-move-constructor)
   RowReader(RowReader&&) = default;