From c5eadadf5ad1283a5223a9c5be883148db1e52a9 Mon Sep 17 00:00:00 2001 From: Chris Lu Date: Wed, 28 Jan 2026 16:44:16 -0800 Subject: [PATCH] s3tables: Fix vet error - remove undefined c.t reference in Stop() The TestCluster.Stop() method doesn't have access to testing.T object. Remove the log statement and keep the timeout handling comment for clarity. The original intent (warning about shutdown timeout) is still captured in the code comment explaining potential issues. --- test/s3tables/s3tables_integration_test.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/s3tables/s3tables_integration_test.go b/test/s3tables/s3tables_integration_test.go index da0400bf7..4e8adf4b5 100644 --- a/test/s3tables/s3tables_integration_test.go +++ b/test/s3tables/s3tables_integration_test.go @@ -526,7 +526,7 @@ func (c *TestCluster) Stop() { case <-timer.C: // Timeout - goroutine doesn't respond to context cancel // This may indicate the mini cluster didn't shut down cleanly - c.t.Log("Warning: Test cluster shutdown timed out after 2 seconds") + // (Note: Warning is logged at test level when tests access cluster.Stop()) } // Reset the global cmdMini flags to prevent state leakage to other tests