• kaa@polly.(none)'s avatar
    Fix for bug #32202: ORDER BY not working with GROUP BY · 8c193678
    kaa@polly.(none) authored
    The bug is a regression introduced by the fix for bug30596. The problem
    was that in cases when groups in GROUP BY correspond to only one row,
    and there is ORDER BY, the GROUP BY was removed and the ORDER BY
    rewritten to ORDER BY <group_by_columns> without checking if the
    columns in GROUP BY and ORDER BY are compatible. This led to
    incorrect ordering of the result set as it was sorted using the
    GROUP BY columns. Additionaly, the code discarded ASC/DESC modifiers
    from ORDER BY even if its columns were compatible with the GROUP BY
    ones.
    
    This patch fixes the regression by checking if ORDER BY columns form a
    prefix of the GROUP BY ones, and rewriting ORDER BY only in that case,
    preserving the ASC/DESC modifiers. That check is sufficient, since the
    GROUP BY columns contain a unique index.
    8c193678
group_by.result 37.5 KB