← Back to team overview

maria-developers team mailing list archive

Re: [Merge] lp:~paul-mccullagh/maria/pbxt-1.0.11 into lp:maria

 

Hi Kristian,

Please go ahead with the patch, its fine with me.

On May 14, 2010, at 8:22 PM, Kristian Nielsen wrote:

Paul McCullagh <paul.mccullagh@xxxxxxxxxxxxx> writes:

Thanks for clearing this up.

The problem is certainly as you say: it depends on the speed of a
background thread.

From this point of view the test does not make all that much sense
for PBXT which returns inconsistent row count estimates.

I think I will adjust the test to work in all cases with PBXT.

We are getting close to ready with MariaDB 5.1.46, so I would like to fix this
issue.

How about this patch, which seems to fix the issue? If you're ok with it, I
will push it into the 5.1-release branch.

- Kristian.

-----------------------------------------------------------------------
=== modified file 'mysql-test/suite/pbxt/r/select_safe.result'
--- mysql-test/suite/pbxt/r/select_safe.result 2009-08-17 15:57:58 +0000 +++ mysql-test/suite/pbxt/r/select_safe.result 2010-05-14 18:19:40 +0000
@@ -1,4 +1,4 @@
-drop table if exists t1;
+drop table if exists t1,t2;
SET SQL_SAFE_UPDATES=1,SQL_SELECT_LIMIT=4, SQL_MAX_JOIN_SIZE=9;
create table t1 (a int auto_increment primary key, b char(20));
insert into t1 values(1,"test");
@@ -13,10 +13,12 @@ a	b
1	test
2	test2
update t1 set b="a" where a=1;
-analyze table t1;
+create table t2 like t1;
+insert into t2 select * from t1;
+analyze table t2;
Table	Op	Msg_type	Msg_text
-test.t1	analyze	status	OK
-select 1 from t1,t1 as t2,t1 as t3;
+test.t2	analyze	status	OK
+select 1 from t2 as t1,t2,t2 as t3;
1
1
1
@@ -77,7 +79,7 @@ id	select_type	table	type	possible_keys	
1	SIMPLE	t1	ALL	b	NULL	NULL	NULL	21	
1	SIMPLE	t2	ref	b	b	21	test.t1.b	1	Using where
SET MAX_SEEKS_FOR_KEY=DEFAULT;
-drop table t1;
+drop table t1, t2;
create table t1 (a int);
insert into t1 values (1),(2),(3),(4),(5);
insert into t1 select * from t1;

=== modified file 'mysql-test/suite/pbxt/t/select_safe.test'
--- mysql-test/suite/pbxt/t/select_safe.test	2009-04-02 10:03:14 +0000
+++ mysql-test/suite/pbxt/t/select_safe.test	2010-05-14 18:19:33 +0000
@@ -3,7 +3,7 @@
#

--disable_warnings
-drop table if exists t1;
+drop table if exists t1,t2;
--enable_warnings

SET SQL_SAFE_UPDATES=1,SQL_SELECT_LIMIT=4, SQL_MAX_JOIN_SIZE=9;
@@ -15,8 +15,13 @@ delete from t1 where a=1;
insert into t1 values(1,"test"),(2,"test2");
SELECT SQL_BUFFER_RESULT * from t1 order by a; # PBXT required for consistent result
update t1 set b="a" where a=1;
-analyze table t1; # PBXT: required to get the correct COUNT(*)
-select 1 from t1,t1 as t2,t1 as t3;
+
+# Test MAX_JOIN_SIZE in a fresh table, as insert/delete combination
+# makes PBXT statistics non-deterministic
+create table t2 like t1;
+insert into t2 select * from t1;
+analyze table t2; # PBXT: required to get the correct COUNT(*)
+select 1 from t2 as t1,t2,t2 as t3;

# The following should give errors:
--error 1175
@@ -64,7 +69,7 @@ set MAX_SEEKS_FOR_KEY=1;
explain select STRAIGHT_JOIN * from t1,t1 as t2 where t1.b=t2.b;
SET MAX_SEEKS_FOR_KEY=DEFAULT;

-drop table t1;
+drop table t1, t2;

# BUG#8726
create table t1 (a int);




--
Paul McCullagh
PrimeBase Technologies
www.primebase.org
www.blobstreaming.org
pbxt.blogspot.com






References