summaryrefslogtreecommitdiff
path: root/sql/table.h
diff options
context:
space:
mode:
authorNeeraj Bisht <neeraj.x.bisht@oracle.com>2013-05-13 17:15:25 +0530
committerNeeraj Bisht <neeraj.x.bisht@oracle.com>2013-05-13 17:15:25 +0530
commit2812634b6c45b96bf3a0def5a6df4267304e8dca (patch)
tree78c6d49a03f1f8f7216fcef2c311d2f47751503b /sql/table.h
parent575559754d957e1119d48dc5cec7d7b94720aa61 (diff)
downloadmariadb-git-2812634b6c45b96bf3a0def5a6df4267304e8dca.tar.gz
Bug#12328597 - MULTIPLE COUNT(DISTINCT) IN SAME SELECT FALSE
WITH COMPOSITE KEY COLUMNS Problem:- While running a SELECT query with several AGGR(DISTINCT) function and these are referring to different field of same composite key, Returned incorrect value. Analysis:- In a table, where we have composite key like (a,b,c) and when we give a query like select COUNT(DISTINCT b), SUM(DISTINCT a) from .... here, we first make a list of items in Aggr(distinct) function (which is a, b), where order of item doesn't matter. and then we see, whether we have a composite key where the prefix of index columns matches the items of the aggregation function. (in this case we have a,b,c). if yes, so we can use loose index scan and we need not perform duplicate removal to distinct in our aggregate function. In our table, we traverse column marked with <-- and get the result as (a,b,c) count(distinct b) sum(distinct a) treated as count b treated as sum(a) (1,1,2)<-- 1 1 (1,2,2)<-- 1++=2 1+1=2 (1,2,3) (2,1,2)<-- 2++=3 1+1+2=4 (2,2,2)<-- 3++=4 1+1+2+2=6 (2,2,3) result will be 4,6, but it should be (2,3) As in this case, our assumption is incorrect. If we have query like select count(distinct a,b), sum(distinct a,b)from .. then we can use loose index scan Solution:- In our query, when we have more then one aggr(distinct) function then they should refer to same fields like select count(distinct a,b), sum(distinct a,b) from .. -->we can use loose scan index as both aggr(distinct) refer to same fields a,b. If they are referring to different field like select count(distinct a), sum(distinct b) from .. -->will not use loose scan index as both aggr(distinct) refer to different fields.
Diffstat (limited to 'sql/table.h')
-rw-r--r--sql/table.h3
1 files changed, 3 insertions, 0 deletions
diff --git a/sql/table.h b/sql/table.h
index 9a85dd83b13..5d4bb654263 100644
--- a/sql/table.h
+++ b/sql/table.h
@@ -894,6 +894,9 @@ enum index_hint_type
INDEX_HINT_FORCE
};
+/* Bitmap of table's fields */
+typedef Bitmap<MAX_FIELDS> Field_map;
+
struct TABLE
{
TABLE() {} /* Remove gcc warning */