aljaz/turso
1
0
Fork 0
mirror of https://github.com/aljazceru/turso.git synced 2026-02-23 17:05:36 +01:00
Code Issues Packages Projects Releases Wiki Activity

optimizer: fix incorrect logic in group_by_contains_all

Browse Source
This commit is contained in:
Jussi Saurio
2025-05-17 17:28:29 +03:00
parent 0fa4ebaec2
commit ce8b2722cf
1 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
core/translate/optimizer/order.rs
View File

@@ -97,10 +97,11 @@ pub fn compute_order_target(
// however in this case we must take the ASC/DESC from ORDER BY into account.
(Some(order_by), Some(group_by)) => {
// Does the group by contain all expressions in the order by?
let group_by_contains_all = group_by.exprs.iter().all(|expr| {
order_by
let group_by_contains_all = order_by.iter().all(|(expr, _)| {
group_by
.exprs
.iter()
.any(|(order_by_expr, _)| exprs_are_equivalent(expr, order_by_expr))
.any(|group_by_expr| exprs_are_equivalent(expr, group_by_expr))
});
// If not, let's try to target an ordering that matches the group by -- we don't care about ASC/DESC
if !group_by_contains_all {