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

sqlite3-parser: box the having clause in GroupBy

Browse Source
This commit is contained in:
Jussi Saurio
2025-02-08 18:08:54 +02:00
parent 2a82091cb3
commit 781aa3b5d6
3 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
core/translate/select.rs
View File

@@ -305,7 +305,7 @@ pub fn prepare_select_plan(
exprs: group_by.exprs,
having: if let Some(having) = group_by.having {
let mut predicates = vec![];
break_predicate_at_and_boundaries(having, &mut predicates);
break_predicate_at_and_boundaries(*having, &mut predicates);
for expr in predicates.iter_mut() {
bind_column_references(
expr,

2
vendored/sqlite3-parser/src/parser/ast/mod.rs
View File

@@ -1005,7 +1005,7 @@ pub struct GroupBy {
/// expressions
pub exprs: Vec<Expr>,
/// `HAVING`
pub having: Option<Expr>, // HAVING clause on a non-aggregate query
pub having: Option<Box<Expr>>, // HAVING clause on a non-aggregate query
}
/// identifier or one of several keywords or `INDEXED`

2
vendored/sqlite3-parser/src/parser/parse.y
View File

@@ -731,7 +731,7 @@ nulls(A) ::= . {A = None;}
%type groupby_opt {Option<GroupBy>}
groupby_opt(A) ::= . {A = None;}
groupby_opt(A) ::= GROUP BY nexprlist(X) having_opt(Y). {A = Some(GroupBy{ exprs: X, having: Y });}
groupby_opt(A) ::= GROUP BY nexprlist(X) having_opt(Y). {A = Some(GroupBy{ exprs: X, having: Y.map(Box::new) });}
%type having_opt {Option<Expr>}
having_opt(A) ::= . {A = None;}