MySQL Lists are EOL. Please join:

List:Commits« Previous MessageNext Message »
From:Alfranio Correia Date:February 3 2009 5:12pm
Subject:Re: bzr commit into mysql-6.0-bugteam branch (horst:3007) Bug#39937
View as plain text  
Hi, horst

It would be great also to remove the timeout from the test.
We should proceed only and only if the number of connections is equal to 1.
If this never happens MTR will eventually kill the test.
What do you think?

I will be online later tonight to review the patch if you need.

Cheers.


horst hunger wrote:
> Hi Rafal,
> alright. Good, that you don't agree. I have another idea, that may fix
> the bug: I wait until the preceding disconnects will be ready. That
> may guarantee that alway the expected 9 queries will be executed
> (without a quit in between, that is cause due to the parallel
> execution of sessions).
> The poposed fix will follow (hopefully) this evening.
> Thanks
> Horst
> Rafal Somla schrieb:


>> Hi Horst,
>>
>> Your solution is based on the assumption that the wrong count (10
>> instead of 9) is caused by extra "Quit" query being counted in. But
>> I'm not totally convinced by this. Looking at the relevant place in
>> the test this is what happens there:
>>
>> ...
>> let $org_queries= `SHOW STATUS LIKE 'Queries'`;
>> SELECT f1();
>> CALL p1();
>> let $new_queries= `SHOW STATUS LIKE 'Queries'`;
>> --disable_log
>> let $diff= `SELECT
>> SUBSTRING('$new_queries',9)-SUBSTRING('$org_queries',9)`;
>> --enable_log
>> eval SELECT $diff;
>> disconnect con1;
>> ...
>>
>> Thus the test assumes that calling f1() and p1() results in 9 queries
>> being executed. How come that "Quit" is also executed? Note that the
>> connection is disconnected *after* getting the $new_queries number.
>>
>> Hence I'm afraid that your patch can hide a real problem we have
>> here. What do you say?
>>
>> Rafal
>>
>> Horst Hunger wrote:
>>> #At file:///work/bzr/mysql-6.0-39937/
>>>
>>>  3007 Horst Hunger    2009-02-03
>>>       Fix for bug#39937: Set the disconnect_timeout to 30 seconds to
>>> have more time in case of lower performance and specified a weaker
>>> condition as sometime also a "Quit" is counted as query.
>>> modified:
>>>   mysql-test/r/status.result
>>>   mysql-test/t/status.test
>>>
>>> === modified file 'mysql-test/r/status.result'
>>> --- a/mysql-test/r/status.result    2009-01-30 14:13:39 +0000
>>> +++ b/mysql-test/r/status.result    2009-02-03 13:30:23 +0000
>>> @@ -231,9 +231,9 @@ f1()
>>>  CALL p1();
>>>  1
>>>  1
>>> -SELECT 9;
>>> -9
>>> -9
>>> +SELECT 9 BETWEEN 9 AND 10;
>>> +9 BETWEEN 9 AND 10
>>> +1
>>>  DROP PROCEDURE p1;
>>>  DROP FUNCTION f1;
>>>  DROP VIEW IF EXISTS v1;
>>>
>>> === modified file 'mysql-test/t/status.test'
>>> --- a/mysql-test/t/status.test    2009-01-30 14:13:39 +0000
>>> +++ b/mysql-test/t/status.test    2009-02-03 13:30:23 +0000
>>> @@ -103,7 +103,7 @@ drop table t1;
>>>  #
>>>  
>>>  # Wait for at most $disconnect_timeout seconds for disconnects to
>>> finish.
>>> -let $disconnect_timeout = 10;
>>> +let $disconnect_timeout = 30;
>>>  
>>>  # Wait for any previous disconnects to finish.
>>>  FLUSH STATUS;
>>> @@ -327,7 +327,8 @@ let $new_queries= `SHOW STATUS LIKE 'Que
>>>  --disable_query_log
>>>  let $diff= `SELECT
>>> SUBSTRING('$new_queries',9)-SUBSTRING('$org_queries',9)`;
>>>  --enable_query_log
>>> -eval SELECT $diff;
>>> +# The number of queries must at least be 9, but ometimes a "Quit"
>>> is also counted.
>>> +eval SELECT $diff BETWEEN 9 AND 10;
>>>  disconnect con1;
>>>  connection default;
>>>  DROP PROCEDURE p1;
>>>
>>>
>
>

Thread
bzr commit into mysql-6.0-bugteam branch (horst:3007) Bug#39937Horst Hunger3 Feb
  • Re: bzr commit into mysql-6.0-bugteam branch (horst:3007) Bug#39937Rafal Somla3 Feb
    • Re: bzr commit into mysql-6.0-bugteam branch (horst:3007) Bug#39937horst hunger5 Feb
    • Re: bzr commit into mysql-6.0-bugteam branch (horst:3007) Bug#39937horst hunger5 Feb
Re: bzr commit into mysql-6.0-bugteam branch (horst:3007) Bug#39937Alfranio Correia3 Feb
  • Re: bzr commit into mysql-6.0-bugteam branch (horst:3007) Bug#39937horst hunger5 Feb
Re: bzr commit into mysql-6.0-bugteam branch (horst:3007) Bug#39937Rafal Somla4 Feb
  • Re: bzr commit into mysql-6.0-bugteam branch (horst:3007) Bug#39937horst hunger5 Feb
  • Re: bzr commit into mysql-6.0-bugteam branch (horst:3007) Bug#39937horst hunger5 Feb
Re: bzr commit into mysql-6.0-bugteam branch (horst:3007) Bug#39937Rafal Somla4 Feb