nodejs-bigtable icon indicating copy to clipboard operation
nodejs-bigtable copied to clipboard

v2.BigtableClient mutateRows: invokes mutateRows with closed client failed

Open flaky-bot[bot] opened this issue 3 years ago • 3 comments

This test failed!

To configure my behavior, see the Flaky Bot documentation.

If I'm commenting on this issue too often, add the flakybot: quiet label and I will stop commenting.


commit: f7fc934631b02c6c1ec3f2fdc8e0e0b160333730 buildURL: Build Status, Sponge status: failed

Test output
Timeout of 10000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/workspace/build/test/gapic_bigtable_v2.js)
Error: Timeout of 10000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/workspace/build/test/gapic_bigtable_v2.js)
    at listOnTimeout (internal/timers.js:554:17)
    at processTimers (internal/timers.js:497:7)

flaky-bot[bot] avatar May 04 '22 07:05 flaky-bot[bot]

Looks like this issue is flaky. :worried:

I'm going to leave this open and stop commenting.

A human should fix and close this.


When run at the same commit (f7fc934631b02c6c1ec3f2fdc8e0e0b160333730), this test passed in one build (Build Status, Sponge) and failed in another build (Build Status, Sponge).

flaky-bot[bot] avatar May 10 '22 07:05 flaky-bot[bot]

Looks like this test is for the generated GAPIC layer of code so this will not be an issue with the veneer.

danieljbruce avatar May 17 '22 14:05 danieljbruce

When searching for the test called invokes mutateRows with closed client you can see that the test uses new bigtableModule.v2.BigtableClient( which is the client in the generated layer and not the client created by the veneer code. Therefore, we should move this issue to the team that maintains the generated layer.

danieljbruce avatar Jun 14 '22 14:06 danieljbruce

As with other issues, https://github.com/googleapis/nodejs-bigtable/pull/1143/files should help.

danieljbruce avatar Feb 09 '23 21:02 danieljbruce