From d660cd6cc36c5ee102abef0f9c533934f41240e4 Mon Sep 17 00:00:00 2001 From: Arthur Schreiber Date: Tue, 7 Oct 2025 15:31:54 +0000 Subject: [PATCH 1/2] Bump the hang detection timeout to 1 second. Signed-off-by: Arthur Schreiber --- go/pools/smartconnpool/pool_test.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/go/pools/smartconnpool/pool_test.go b/go/pools/smartconnpool/pool_test.go index 27aa8a2300b..80196cb6eb0 100644 --- a/go/pools/smartconnpool/pool_test.go +++ b/go/pools/smartconnpool/pool_test.go @@ -1247,7 +1247,7 @@ func TestGetSpike(t *testing.T) { func TestCloseDuringWaitForConn(t *testing.T) { ctx := context.Background() goRoutineCnt := 50 - getTimeout := 300 * time.Millisecond + getTimeout := 1000 * time.Millisecond for range 50 { hung := make(chan (struct{}), goRoutineCnt) @@ -1263,7 +1263,7 @@ func TestCloseDuringWaitForConn(t *testing.T) { wg := sync.WaitGroup{} var count atomic.Int64 - fmt.Println("Starting TestCloseDuringGetAndPut") + fmt.Println("Starting TestCloseDuringWaitForConn") // Spawn multiple goroutines to perform Get and Put operations, but only // allow connections to be checked out until `closed` has been set to true. From 2054c80bb5b0341c1b0a5d2d7e5401453b0b55b8 Mon Sep 17 00:00:00 2001 From: Arthur Schreiber Date: Tue, 7 Oct 2025 16:29:48 +0000 Subject: [PATCH 2/2] Bump the timeout to 2 seconds just to be sure. Signed-off-by: Arthur Schreiber --- go/pools/smartconnpool/pool_test.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/go/pools/smartconnpool/pool_test.go b/go/pools/smartconnpool/pool_test.go index 80196cb6eb0..24100864d25 100644 --- a/go/pools/smartconnpool/pool_test.go +++ b/go/pools/smartconnpool/pool_test.go @@ -1247,7 +1247,7 @@ func TestGetSpike(t *testing.T) { func TestCloseDuringWaitForConn(t *testing.T) { ctx := context.Background() goRoutineCnt := 50 - getTimeout := 1000 * time.Millisecond + getTimeout := 2000 * time.Millisecond for range 50 { hung := make(chan (struct{}), goRoutineCnt)