Skip to content

Commit

Permalink
CI: fix to Online DDL flow test, do not t.Log from within a goroutine (
Browse files Browse the repository at this point in the history
…#17496)

Signed-off-by: Shlomi Noach <[email protected]>
  • Loading branch information
shlomi-noach authored Jan 9, 2025
1 parent b865d13 commit aabf1c9
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions go/test/endtoend/onlineddl/flow/onlineddl_flow_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -244,7 +244,7 @@ func TestOnlineDDLFlow(t *testing.T) {
select {
case <-ticker.C:
case <-workloadCtx.Done():
t.Logf("Terminating routine throttler check")
fmt.Println("Terminating routine throttler check")
return
}
}
Expand All @@ -258,8 +258,8 @@ func TestOnlineDDLFlow(t *testing.T) {
wg.Add(1)
go func() {
defer cancel()
defer t.Logf("Terminating workload")
defer wg.Done()
defer fmt.Println("Terminating workload")
runMultipleConnections(workloadCtx, t)
}()
})
Expand Down

0 comments on commit aabf1c9

Please sign in to comment.