~drizzle-trunk/drizzle/development

  • Committer: Stewart Smith
  • Date: 2010-08-12 16:48:46 UTC
  • Revision ID: stewart@flamingspork.com-20100812164846-s9bhy47g60bvqs41
bug lp:611379 Equivalent queries with Impossible where return different results

The following two equivalent queries return different results in maria 5.2 and 5.3 (and identical results in mysql 5.5.5) :

SELECT SUM( DISTINCT table1 .`pk` ) FROM B table1 STRAIGHT_JOIN ( BB table2 JOIN CC ON table2 .`col_varchar_key` ) ON table2 .`pk` ;

SELECT * FROM ( SELECT SUM( DISTINCT table1 .`pk` ) FROM B table1 STRAIGHT_JOIN ( BB table2 JOIN CC ON table2 .`col_varchar_key` ) ON table2 .`pk` );

MariaDB returns 0 on the second query and NULL on the first, whereas MySQL returns NULL on both. In MariaDB, both EXPLAIN plans agree that "Impossible WHERE noticed after reading const tables"



We have some slightly different output in drizzle:

main.bug_lp611379 [ fail ]
drizzletest: At line 9: query 'explain select * from (select sum(distinct t1.a) from t1,t2 where t1.a=t2.a)
as t' failed: 1048: Column 'sum(distinct t1.a)' cannot be null

but the fix gets us the correct query results, although with slightly different execution plans.



This fix is directly ported from MariaDB.
Filename Latest Rev Last Changed Committer Comment Size
..
tests 1340.1.2 15 years ago Brian Aker Merge of table cache/def DD. Diff
dictionary.cc 1530.2.6 14 years ago Monty Taylor Moved plugin::Context to module::Context. Fixed va 1.6 KB Diff Download File
dictionary.h 1603 14 years ago Brian Aker Remove current_session from usage in table stats l 1.3 KB Diff Download File
plugin.ini 1460 15 years ago Brian Aker Seeing static effects valgrind output 300 bytes Diff Download File
table_cache.cc 1689.2.7 14 years ago Brian Aker LOCK_open to boost. 3.7 KB Diff Download File
table_cache.h 1340.1.2 15 years ago Brian Aker Merge of table cache/def DD. 1.6 KB Diff Download File
table_definition_cache.cc 1689.2.7 14 years ago Brian Aker LOCK_open to boost. 2.9 KB Diff Download File
table_definition_cache.h 1618 14 years ago Brian Aker This is a rollup set of patches for modifications 1.6 KB Diff Download File