google-cloud-cpp

google-cloud-cpp copied to clipboard

Supporting Bazel v7 (even without bzlmod) will require multiple steps. We will use this issue to discuss the ideas and then use it as a tracking bug. ## Background At...

coryan

Add any context about the cleanup here.

alevenberg

Add LRO error handling for operations not using `google::longrunning::Operation`

3

If an error has occurred, we need to correctly interrogate the operation type in the response and construct an appropriate `Status` object.

scotthart

Consider combining bool is_map and is_message fields into an enum field

1

Determine if combining these fields in `DiscoveryTypeVertex::TypeInfo` makes sense and if it increases readability or not.

scotthart

Determine if we can stop emitting request field annotation in Discovery protos

1

Now that `google.api.http` handling has improved and generated protos are correctly populating the `body` field, determine if we still need to continue annotating the same field listed as the value...

scotthart

Consider tracking method comment substitutions

1

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...

dbolduc

de-dup Spanner logging of streaming-RPC responses

1

When Spanner started using generated stubs (see #11854 and #11888) it picked up generic support for streaming-RPC logging via `spanner_internal::SpannerLogging` (when the "rpc" tracing component is enabled), which adds `internal::StreamingReadRpcLogging`...

devbww

scotthart

scotthart

Implement AsyncClient::ListObjects() with streaming

1

coryan