Darren Bolduc

Results 144 comments of Darren Bolduc

Seems like other client library languages do full jitter only. Given the supposed, slight performance benefit, I think we should implement min-jitter (which is like full jitter, but slightly more...

We don't have time to do this. We are focusing on the async client.

* GCB: https://console.cloud.google.com/cloud-build/builds;region=us-east1/22a81e5a-a17f-4180-89d5-a96488bfd4a9;tab=detail?project=cloud-cpp-testing-resources * Raw: https://storage.googleapis.com/cloud-cpp-community-publiclogs/logs/google-cloud-cpp/main/269a09a280745cf109890a247578921b7d524f11/fedora-34-coverage/log-22a81e5a-a17f-4180-89d5-a96488bfd4a9.txt

Just had a crash on a `cxx-20-pr`: * GCB: https://console.cloud.google.com/cloud-build/builds;region=us-east1/8c481c69-8225-48a9-ae54-bb9488c52978;tab=detail?project=cloud-cpp-testing-resources * Raw: https://storage.googleapis.com/cloud-cpp-community-publiclogs/logs/google-cloud-cpp/10185/d94cb2867b202edc9b818fec01829836aa050d7e/fedora-36-cxx20-cxx20/log-8c481c69-8225-48a9-ae54-bb9488c52978.txt And there was the following from 2022-10-20 (also captured in #10082): * GCB: https://console.cloud.google.com/cloud-build/builds;region=us-east1/7c25362c-3036-4279-8788-0abe9b9bdca0;tab=detail?project=cloud-cpp-testing-resources * Raw: https://storage.googleapis.com/cloud-cpp-community-publiclogs/logs/google-cloud-cpp/main/c8625e063050fc406de8a5afe7913a9a1d80b082/fedora-36-cmake-gcs-rest/log-7c25362c-3036-4279-8788-0abe9b9bdca0.txt

* GCB: https://console.cloud.google.com/cloud-build/builds;region=us-east1/796f25ed-1a06-4be1-b41a-df9eb87ddeb5;tab=detail?project=cloud-cpp-testing-resources * Raw: https://storage.googleapis.com/cloud-cpp-community-publiclogs/logs/google-cloud-cpp/main/4b1a9a852840542d67ec427b36a888a39d744f00/fedora-36-cxx20-cxx20/log-796f25ed-1a06-4be1-b41a-df9eb87ddeb5.txt `cxx-20` again

we jinxed it * GCB: https://console.cloud.google.com/cloud-build/builds;region=us-east1/37833209-515b-443f-8d5f-3c09c3dc3f85;tab=detail?project=cloud-cpp-testing-resources * Raw: https://storage.googleapis.com/cloud-cpp-community-publiclogs/logs/google-cloud-cpp/main/3368e3a1a5c9adbcd6e392e558740abb5d93b744/fedora-37-cxx20-cxx20/log-37833209-515b-443f-8d5f-3c09c3dc3f85.txt

While not the Pub/Sub expert, let me jump in and confirm a few of your suspicions. > Each call of `MakePublisherConnection` is still tied to a topic. Yes. > It...

> Perhaps that could be discussed among maintainers for a future release? Definitely. I will start the brainstorming of ways for the fancy `Publisher` to support sharing gRPC channel pools......

> "never working" but the script successfully connected? Good point. > is it likely that the first call was a `DeleteTable`? Pay no mind to `DeleteTable`. I copy/pasted a random...

* GCB: https://console.cloud.google.com/cloud-build/builds;region=us-east1/e76a861c-74d0-4bd3-9051-44355a6b341a;tab=detail?project=cloud-cpp-testing-resources * Raw: https://storage.googleapis.com/cloud-cpp-community-publiclogs/logs/google-cloud-cpp/main/91f2f180b071b4db92ea936b2b28729a4f904fad/fedora-latest-bazel-tsan/log-e76a861c-74d0-4bd3-9051-44355a6b341a.txt