Skip to content

Commit 2b4ac42

Browse files
authored
fix(bigtable): retry unexpected EOF errors (#13157)
This has been previously discussed at #11276. I am resubmitting the change again, since we still see the UnexpectedEOF and we are running with this patch in production to retry it. Apparently there is a flaky test that shows the same issue as well: #10909 There are more cases that use the comparison of the error message now, and the grpc-go has [some places](https://github.com/grpc/grpc-go/blob/ae62635cb0e21ae37e7ae5f0f18aaac5e8139834/rpc_util.go#L1033) that return a gRPC status error with the error message of the io.UnexpectedEOF. So I kept the handling by error message text instead of comparing directly to io.UnexpectedEOF.
1 parent 65c510f commit 2b4ac42

1 file changed

Lines changed: 5 additions & 0 deletions

File tree

bigtable/bigtable.go

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -64,6 +64,11 @@ var (
6464
"Received Rst stream",
6565
"RST_STREAM closed stream",
6666
"Received RST_STREAM",
67+
68+
// https://github.com/googleapis/google-cloud-go/issues/10207#issuecomment-2604859656
69+
// https://github.com/googleapis/google-cloud-go/issues/10909
70+
// https://github.com/googleapis/google-cloud-go/pull/11276
71+
"unexpected EOF",
6772
}
6873
defaultBackoff = gax.Backoff{
6974
Initial: 100 * time.Millisecond,

0 commit comments

Comments
 (0)