Noah Dietz comments

Results 127 comments of


                                            Noah Dietz

pubsub: With exactly once delivery and message ordering enabled, message order is not mantained

triaged labels, feel free to change

chore(all): update opentelemetry-go monorepo to v1.23.1

@quartzmo assigning to you to eval. LMK if we are just in the clear with otel deps these days and I will handle them accordingly. Not wanting to step on...

all: consider a way to automatically Close clients

Sounds like a great idea. We are already half way there - we have a wrapper Client type with the transport-specific `internalClient` interface as a field. The integration tests in...

profiler: high memory usage leading to OOM

triaged labels, feel free to change

fix(datastore): propagate ctx on query error

@alon-ne hey thanks for the PR. Have you opened an issue yet? If not, please do so and include a repro if possible! Also, if there is a test that...

spanner: allow to bind spanner.Statement to a placeholder

triaged labels, feel free to change

spanner: cannot find module providing package cel.dev/expr

@sergiitk I'm just here triaging bugs, will this be fixed with a new `xds` release/dependency update?

pubsub: move off of dependency on go.einride.tech/aip

@hongalex if we can verify that this is a test only dep and it doesn't end up in the client's compile library, then it can stay.

pubsub: move off of dependency on go.einride.tech/aip

Ok, thanks Alex for taking the time to validate things. We can keep it for the time being, I'll close this.

logging: OpenTelemetry spans are ignored as parents for LogRecords

It sounds like there is some unexpected behavior re:tracing contexts used so I added the `type: bug` label as part of issue triage, feel free to change that, just trying...