From: Bjorn Munch
Date: April 7 2011 8:32am
Subject: Re: bzr commit into mysql-next-mr-wl5569 branch
 (andrei.elkin:3274) WL#5569
List-Archive: http://lists.mysql.com/commits/134896
Message-Id: <20110407083213.GA7911@khepri15.norway.sun.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii

On 05/04 16.26, Andrei Elkin wrote:
> #At file:///home/andrei/MySQL/BZR/2a-23May/WL/mysql-next-mr-wl5569/ based on revid:andrei.elkin@stripped
> 
>  3274 Andrei Elkin	2011-04-05
>       wl#5569 MTS
>       
>       restoring previous 4 default workers that rpl_parallel works with.
> 
>     modified:
>       mysql-test/extra/rpl_tests/rpl_parallel_load.test
>       mysql-test/suite/rpl/r/rpl_parallel.result
>       mysql-test/suite/rpl/t/rpl_parallel-slave.opt
> === modified file 'mysql-test/extra/rpl_tests/rpl_parallel_load.test'
> --- a/mysql-test/extra/rpl_tests/rpl_parallel_load.test	2011-04-03 10:07:30 +0000
> +++ b/mysql-test/extra/rpl_tests/rpl_parallel_load.test	2011-04-05 16:26:37 +0000
> @@ -260,26 +260,26 @@ select time_to_sec(@m_1) - time_to_sec(@
>         time_to_sec(@s_1) - time_to_sec(@s_0) as 'delta_s' into outfile 'delta.out';
>  
>  
> -# let $i = $databases + 1;
> -# while($i)
> -# {
> -#   let $i1 = $i;
> -#   dec $i1;
> -
> -#   let $diff_tables=master:test$i1.v_tm_nk, slave:test$i1.v_tm_nk;
> -#   source include/diff_tables.inc;
> -
> -#   let $diff_tables=master:test$i1.v_ti_nk, slave:test$i1.v_ti_nk;
> -#   source include/diff_tables.inc;
> +let $i = $databases + 1;

let $i = $databases + 1;

This does not do any arithmetic, is creates a *string* ending in
" + 1". When you try do use this as a number later, that " + 1" is
probably ignored.

This particular arithmetic can be achieved thus:

let $i = $databases;
inc $i;

In the more general case you'd have to resort to

let $var= `SELECT <some expression>`;

Maybe this wasn't your code originally, but I'd like to point it out
anyway.

- Bjorn