impl(v3): decommission PublisherOptions and SubscriberOptions (#15891)

Author: scotthart

ci/abi-dumps/google_cloud_cpp_pubsub.expected.abi.dump.gz

@@ -292,6 +292,117 @@ internal legacy files.
 
 ### Pubsub
 
+<details>
+  <summary>Removed <code>pubsub::PublisherOptions</code>
+</summary>
+#### `pubsub::PublisherOptions`
+
+The deprecated `pubsub::PublisherOptions` has been removed. Please use
+`google::cloud::Options` instead.
+
+The following table shows the mapping from `pubsub::PublisherOptions` methods to
+their `google::cloud::Options` equivalents:
+
+| `pubsub::PublisherOptions` method | `google::cloud::Options` equivalent                |
+| --------------------------------- | -------------------------------------------------- |
+| `(constructor)`                   | `google::cloud::Options{}`                         |
+| `set_maximum_hold_time`           | `google::cloud::pubsub::MaxHoldTimeOption`         |
+| `set_maximum_batch_message_count` | `google::cloud::pubsub::MaxBatchMessagesOption`    |
+| `set_maximum_batch_bytes`         | `google::cloud::pubsub::MaxBatchBytesOption`       |
+| `enable_message_ordering`         | `google::cloud::pubsub::MessageOrderingOption`     |
+| `disable_message_ordering`        | `google::cloud::pubsub::MessageOrderingOption`     |
+| `set_maximum_pending_bytes`       | `google::cloud::pubsub::MaxPendingBytesOption`     |
+| `set_maximum_pending_messages`    | `google::cloud::pubsub::MaxPendingMessagesOption`  |
+| `set_full_publisher_ignored`      | `google::cloud::pubsub::FullPublisherActionOption` |
+| `set_full_publisher_rejects`      | `google::cloud::pubsub::FullPublisherActionOption` |
+| `set_full_publisher_blocks`       | `google::cloud::pubsub::FullPublisherActionOption` |
+
+Example usage of the replacements can be found below.
+
+**Before:**
+
+```cpp
+namespace gc = ::google::cloud;
+namespace pubsub = ::google::cloud::pubsub;
+
+auto publisher_options = pubsub::PublisherOptions{}
+    .enable_message_ordering()
+    .set_full_publisher_ignored();
+
+auto publisher = pubsub::Publisher(pubsub::MakePublisherConnection(
+    topic, publisher_options);
+```
+
+**After:**
+
+```cpp
+namespace gc = ::google::cloud;
+namespace pubsub = ::google::cloud::pubsub;
+
+auto options = gc::Options{}
+    .set<pubsub::MessageOrderingOption>(true)
+    .set<pubsub::FullPublisherActionOption>(
+        pubsub::FullPublisherAction::kIgnored);
+
+auto publisher = pubsub::Publisher(pubsub::MakePublisherConnection(
+    topic, options));
+```
+
+</details>
+
+<details>
+  <summary>Removed <code>pubsub::SubscriberOptions</code>
+</summary>
+#### `pubsub::SubscriberOptions`
+
+The deprecated `pubsub::SubscriberOptions` has been removed. Please use
+`google::cloud::Options` instead.
+
+The following table shows the mapping from `pubsub::SubscriberOptions` methods
+to their `google::cloud::Options` equivalents:
+
+| `pubsub::SubscriberOptions` method | `google::cloud::Options` equivalent                   |
+| ---------------------------------- | ----------------------------------------------------- |
+| `(constructor)`                    | `google::cloud::Options{}`                            |
+| `set_max_deadline_time`            | `google::cloud::pubsub::MaxDeadlineTimeOption`        |
+| `set_max_deadline_extension`       | `google::cloud::pubsub::MaxDeadlineExtensionOption`   |
+| `set_max_outstanding_messages`     | `google::cloud::pubsub::MaxOutstandingMessagesOption` |
+| `set_max_outstanding_bytes`        | `google::cloud::pubsub::MaxOutstandingBytesOption`    |
+| `set_max_concurrency`              | `google::cloud::pubsub::MaxConcurrencyOption`         |
+| `set_shutdown_polling_period`      | `google::cloud::pubsub::ShutdownPollingPeriodOption`  |
+
+Example usage of the replacements can be found below.
+
+**Before:**
+
+```cpp
+namespace gc = ::google::cloud;
+namespace pubsub = ::google::cloud::pubsub;
+
+auto subscriber_options = pubsub::SubscriberOptions{}
+    .set_max_deadline_time(std::chrono::seconds(10))
+    .set_max_outstanding_messages(42);
+
+auto subscriber = pubsub::Subscriber(pubsub::MakeSubscriberConnection(
+    subscription, subscriber_options);
+```
+
+**After:**
+
+```cpp
+namespace gc = ::google::cloud;
+namespace pubsub = ::google::cloud::pubsub;
+
+auto options = gc::Options{}
+    .set<pubsub::MaxDeadlineTimeOption>(std::chrono::seconds(10))
+    .set<pubsub::MaxOutstandingMessagesOption>(42);
+
+auto subscriber = pubsub::Subscriber(pubsub::MakeSubscriberConnection(
+    subscription, options));
+```
+
+</details>
+
 ### Spanner
 
 <details>

doc/v3-migration-guide.md

@@ -243,8 +243,6 @@ add_library(
     publisher.h
     publisher_connection.cc
     publisher_connection.h
-    publisher_options.cc
-    publisher_options.h
     pull_ack_handler.cc
     pull_ack_handler.h
     pull_response.h
@@ -266,8 +264,6 @@ add_library(
     subscriber.h
     subscriber_connection.cc
     subscriber_connection.h
-    subscriber_options.cc
-    subscriber_options.h
     subscription.cc
     subscription.h
     subscription_admin_client.cc
@@ -494,14 +490,12 @@ function (google_cloud_cpp_pubsub_client_define_tests)
         message_test.cc
         options_test.cc
         publisher_connection_test.cc
-        publisher_options_test.cc
         publisher_test.cc
         pull_ack_handler_test.cc
         schema_test.cc
         snapshot_builder_test.cc
         snapshot_test.cc
         subscriber_connection_test.cc
-        subscriber_options_test.cc
         subscriber_test.cc
         subscription_admin_client_test.cc
         subscription_admin_connection_test.cc

google/cloud/pubsub/CMakeLists.txt

@@ -14,6 +14,7 @@
 
 #include "google/cloud/pubsub/admin/subscription_admin_client.h"
 #include "google/cloud/pubsub/admin/topic_admin_client.h"
+#include "google/cloud/pubsub/options.h"
 #include "google/cloud/pubsub/publisher.h"
 #include "google/cloud/pubsub/subscriber.h"
 #include "google/cloud/pubsub/testing/random_names.h"

google/cloud/pubsub/benchmarks/throughput.cc

@@ -16,7 +16,6 @@
 #define GOOGLE_CLOUD_CPP_GOOGLE_CLOUD_PUBSUB_BLOCKING_PUBLISHER_H
 
 #include "google/cloud/pubsub/blocking_publisher_connection.h"
-#include "google/cloud/pubsub/publisher_options.h"
 #include "google/cloud/pubsub/version.h"
 #include <string>
 

google/cloud/pubsub/blocking_publisher.h

@@ -19,7 +19,6 @@
 #include "google/cloud/pubsub/connection_options.h"
 #include "google/cloud/pubsub/internal/publisher_stub.h"
 #include "google/cloud/pubsub/message.h"
-#include "google/cloud/pubsub/publisher_options.h"
 #include "google/cloud/pubsub/retry_policy.h"
 #include "google/cloud/pubsub/topic.h"
 #include "google/cloud/pubsub/version.h"

google/cloud/pubsub/blocking_publisher_connection.h

@@ -129,7 +129,6 @@ google_cloud_cpp_pubsub_hdrs = [
     "options.h",
     "publisher.h",
     "publisher_connection.h",
-    "publisher_options.h",
     "pull_ack_handler.h",
     "pull_response.h",
     "retry_policy.h",
@@ -142,7 +141,6 @@ google_cloud_cpp_pubsub_hdrs = [
     "snapshot_builder.h",
     "subscriber.h",
     "subscriber_connection.h",
-    "subscriber_options.h",
     "subscription.h",
     "subscription_admin_client.h",
     "subscription_admin_connection.h",
@@ -248,7 +246,6 @@ google_cloud_cpp_pubsub_srcs = [
     "message.cc",
     "options.cc",
     "publisher_connection.cc",
-    "publisher_options.cc",
     "pull_ack_handler.cc",
     "schema.cc",
     "schema_client.cc",
@@ -258,7 +255,6 @@ google_cloud_cpp_pubsub_srcs = [
     "snapshot_builder.cc",
     "subscriber.cc",
     "subscriber_connection.cc",
-    "subscriber_options.cc",
     "subscription.cc",
     "subscription_admin_client.cc",
     "subscription_admin_connection.cc",

google/cloud/pubsub/google_cloud_cpp_pubsub.bzl

@@ -21,6 +21,7 @@
 #include "google/cloud/pubsub/internal/noop_message_callback.h"
 #include "google/cloud/pubsub/internal/streaming_subscription_batch_source.h"
 #include "google/cloud/pubsub/internal/subscriber_stub_factory.h"
+#include "google/cloud/pubsub/options.h"
 #include "google/cloud/pubsub/publisher.h"
 #include "google/cloud/pubsub/subscriber.h"
 #include "google/cloud/pubsub/subscription.h"

google/cloud/pubsub/integration_tests/subscriber_integration_test.cc

@@ -13,6 +13,7 @@
 // limitations under the License.
 
 #include "google/cloud/pubsub/internal/batching_publisher_connection.h"
+#include "google/cloud/pubsub/options.h"
 #include "google/cloud/internal/make_status.h"
 
 namespace google {

google/cloud/pubsub/internal/batching_publisher_connection.cc

@@ -14,6 +14,7 @@
 
 #include "google/cloud/pubsub/internal/batching_publisher_connection.h"
 #include "google/cloud/pubsub/internal/defaults.h"
+#include "google/cloud/pubsub/options.h"
 #include "google/cloud/pubsub/testing/mock_batch_sink.h"
 #include "google/cloud/future.h"
 #include "google/cloud/internal/random.h"