Megan Potter comments

Results 189 comments of


                                            Megan Potter

trafficstars

Split from 1425: I also have a few other questions, if someone can help me will really appreciate it

Closing for inactivity.

Cannot update `ackDeadline` via `setOptions()`

Ping, this is still on the radar.

Unhandled promises in Topic.flush and PubSub.close prevent proper shutdown

This got some changes from the exactly once delivery work, and it looks like the current unit tests should cover it, so I think this is finished. Please comment if...

modifyAckDeadline fails with Request payload size exceeds the limit: 524288 bytes when it gets called with too many messages

We have moved this issue to our internal tracking system. However, the issue is considered low-priority at this time. If the issue is ultimately resolved, it will appear in the...

"Failed to connect before the deadline" error

Ping, the meta-issue above is still under investigation.

Publishing messages burst timeout

@ForbesLindesay That's unfortunately an internal bug reference. We're trying to build some momentum for cross-functional debugging on some of these issues, because there's a belief that it involves several components...

Calling `.nack()` disables exponential backoff and creates log jams

I think we had a discussion about this before and that had a "works as expected" sort of result, but I'll try to get some attention on this again. @kamalaboulhosn...

Add samples for RetryPolicy and filtering

@greensopinion Thanks for the gist. I think we're going to add a sample for this after all. There was some debate about whether the [sample browser](https://cloud.google.com/docs/samples?language=nodejs&product=pubsub) site was for any...

Google Cloud Pub/Sub triggers high latency on low messages throughput

Hi all, I just made some updates to the batch flushing logic for publishing, and that's in 3.4.0. I'd be interested to hear if it helps this.

OpenTelemetry integration misses the point: we need to propagate the span/context!

Mentioned in the PR, but mentioning here too: there's a suuuuper early preview branch now if anyone wants to play around with what's done. I don't recommend using it in...