Darren Bolduc
Darren Bolduc
We already do this for custom retry, idempotency policies. Custom `PollingPolicy`s should not be so hard. We need to write the sample: https://github.com/googleapis/google-cloud-cpp/blob/main/generator/internal/sample_generator.cc Update the scaffold with words about the...
It would be ideal if the trace exporter does not block the applications. We would need to generate an async API for `trace_v2::TraceClient::AsyncBatchWriteSpans` We would need to hold the futures...
* GCB: https://console.cloud.google.com/cloud-build/builds;region=us-east1/e77ca00f-99ba-49d8-bc8e-997f2f116ed5;tab=detail?project=cloud-cpp-testing-resources * Raw: https://storage.googleapis.com/cloud-cpp-community-publiclogs/logs/google-cloud-cpp/11897/2d7c0e227f4347bf7775900139b507f641acc1df/fedora-37-bazel-libcxx/log-e77ca00f-99ba-49d8-bc8e-997f2f116ed5.txt Seems like everything was hunky-dory until we launched the Bigtable emulator and no tests could connect to it. ``` 2023-08-09T19:19:11Z (+3950s): Launching Cloud Bigtable...
In #13546 we made `bigtable_mocks` header only. We did this to work around a problem. CMake complained when we tried to export `google_cloud_cpp_bigtable_mocks` because it cannot find the linked library:...
aip/4222 says: >If the [google.api.http](https://github.com/googleapis/googleapis/blob/master/google/api/http.proto) annotation contains additional_bindings, these patterns must be parsed for additional request parameters. Fields not duplicated in the top-level (or additional_bindings) pattern must be included in...
We have a system for tracking whether substitutions that we make on parameter comments from protos are used. https://github.com/googleapis/google-cloud-cpp/blob/7c4bd275d07df0a8a3652c1bd40038ea5d434818/generator/internal/descriptor_utils.cc#L463-L465 https://github.com/googleapis/google-cloud-cpp/blob/7c4bd275d07df0a8a3652c1bd40038ea5d434818/generator/internal/descriptor_utils.cc#L600 We should consider doing the same thing for substitutions that...
We have defaults, but as @coryan points out they should be configurable by the application. Applications not running on GCP will want to disable any retrying. Applications running on GKE...
https://cloud.google.com/cpp/docs/reference/bigtable/latest/classgoogle_1_1cloud_1_1bigtable_1_1ExponentialBackoffPolicy#classgoogle_1_1cloud_1_1bigtable_1_1ExponentialBackoffPolicy_1a77a0d66025cd00fcb33a20d7ce5d562c We have a public constructor using an internal type. We should probably just move it to private. I think that is allowed. Also, the legacy retry classes should be...
Bigtable returns a trailer: `x-goog-ext-425905942-bin`, which is the following proto encoded: ```proto message RequestParams { optional string cluster_zone = 1; optional string cluster_id = 2; } ``` This is the...
We have separate libraries for Dialogflow v2 and v3. - `google/cloud/dialogflow_cx` - `google/cloud/dialogflow_es` These were generated before we established the convention that one library should hold all versions of a...