address gemini review comments

Author: scotthart

google/cloud/bigtable/data_connection_test.cc

@@ -74,25 +74,6 @@ TEST(MakeDataConnection, TracingEnabled) {
               Contains(SpanNamed("bigtable::Table::Apply")));
 }
 
-TEST(MakeDataConnection, InstanceChannelAffinityOption) {
-  InstanceResource instance_a{Project("my-project"), "instance-a"};
-  InstanceResource instance_b{Project("my-project"), "instance-b"};
-  auto conn = MakeDataConnection(
-      TestOptions()
-          .set<AppProfileIdOption>("user-supplied")
-          .set<bigtable_internal::InstanceChannelAffinityOption>(
-              {instance_a, instance_b}));
-  auto options = conn->options();
-  EXPECT_TRUE(options.has<DataBackoffPolicyOption>())
-      << "Options are not defaulted in MakeDataConnection()";
-  EXPECT_EQ(options.get<AppProfileIdOption>(), "user-supplied")
-      << "User supplied Options are overridden in MakeDataConnection()";
-  ASSERT_TRUE(options.has<bigtable_internal::InstanceChannelAffinityOption>());
-  EXPECT_THAT(
-      options.get<bigtable_internal::InstanceChannelAffinityOption>().size(),
-      Eq(2));
-}
-
 TEST(MakeDataConnection, TracingDisabled) {
   auto span_catcher = testing_util::InstallSpanCatcher();
 

google/cloud/bigtable/options.h

@@ -180,8 +180,8 @@ struct QueryPlanRefreshFunctionRetryPolicyOption {
  * If `MakeDataConnection(std::vector<InstanceResource>, Options)` is called,
  * then all connections will be managed by a Dynamic Channel Pool. The
  * `DynamicChannelPoolSizingPolicy` can be provided via the
- * `DynamicChannelPoolSizingPolicyOption` and configures the behavior of the
- * `DynamicChannelPool`.
+ * `experimental::DynamicChannelPoolSizingPolicyOption` and configures the
+ * behavior of the `DynamicChannelPool`.
  */
 struct DynamicChannelPoolSizingPolicy {
   // Removing unused channels is not as performance critical as adding channels