• Jorgen Loland's avatar
    Bug#47280 - strange results from count(*) with order by multiple · 6da93b22
    Jorgen Loland authored
                columns without where/group
                         
    Simple SELECT with implicit grouping used to return many rows if
    the query was ordered by the aggregated column in the SELECT
    list. This was incorrect because queries with implicit grouping
    should only return a single record.
                                  
    The problem was that when JOIN:exec() decided if execution needed
    to handle grouping, it was assumed that sum_func_count==0 meant
    that there were no aggregate functions in the query. This
    assumption was not correct in JOIN::exec() because the aggregate
    functions might have been optimized away during JOIN::optimize().
                      
    The reason why queries without ordering behaved correctly was
    that sum_func_count is only recalculated if the optimizer chooses
    to use temporary tables (which it does in the ordered case).
    Hence, non-ordered queries were correctly treated as grouped.
                      
    The fix for this bug was to remove the assumption that
    sum_func_count==0 means that there is no need for grouping. This
    was done by introducing variable "bool implicit_grouping" in the
    JOIN object.
    
    mysql-test/r/func_group.result:
      Add test for BUG#47280
    mysql-test/t/func_group.test:
      Add test for BUG#47280
    sql/opt_sum.cc:
      Improve comment for opt_sum_query()
    sql/sql_class.h:
      Add comment for variables in TMP_TABLE_PARAM
    sql/sql_select.cc:
      Introduce and use variable implicit_grouping instead of (!group_list && sum_func_count) in places that need to test if grouping is required. Also added comments for: optimization of aggregate fields for implicitly grouped queries  (JOIN::optimize) and choice of end_select method (JOIN::execute)
    sql/sql_select.h:
      Add variable implicit_grouping, which will be TRUE for queries that contain aggregate functions but no GROUP BY clause. Also added comment to sort_and_group variable.
    6da93b22
func_group.test 32.5 KB