From 0267cad5e2a07ceb9ef12384d853de8140db0108 Mon Sep 17 00:00:00 2001 From: Sergei Golubchik Date: Wed, 13 Sep 2017 22:45:20 +0200 Subject: [PATCH] bugfix: mtr non-reliable failure detection The $result variable was supposed to accumulate failures from all mysqltest processes (that analyzed all error logs in parallel). But it was mistakenly put in the loop, so it was reset for each process. --- mysql-test/mysql-test-run.pl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mysql-test/mysql-test-run.pl b/mysql-test/mysql-test-run.pl index d760efc3744..5589e74a998 100755 --- a/mysql-test/mysql-test-run.pl +++ b/mysql-test/mysql-test-run.pl @@ -4570,8 +4570,8 @@ sub check_warnings ($) { my $timeout= start_timer(check_timeout($tinfo)); + my $result= 0; while (1){ - my $result= 0; my $proc= My::SafeProcess->wait_any_timeout($timeout); mtr_report("Got $proc");