aljaz/turso
1
0
Fork 0
mirror of https://github.com/aljazceru/turso.git synced 2025-12-28 21:44:21 +01:00
Code Issues Packages Projects Releases Wiki Activity

Add comment

Browse Source
This commit is contained in:
jussisaurio
2024-08-01 17:40:39 +03:00
parent 583fe31667
commit 81c2f2eca6
1 changed files with 5 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
core/translate/where_clause.rs
View File

@@ -277,6 +277,11 @@ pub fn translate_processed_where<'a>(
// and end up with a result where u.id = 3 and p.id = 5, which is incorrect.
// Instead we replace the second SeekRowid with a comparison against the row that was already fetched,
// i.e. we compare p.id == 5, which would not match (and is the correct result).
//
// It would probably be better to modify the AST in the WhereTerms directly, but that would require
// refactoring to not use &'a Ast::Expr references in the WhereTerms, i.e. the WhereClause would own its data
// and could mutate it to change the query as needed. We probably need to do this anyway if we want to have some
// kind of Query Plan construct that is not just a container for AST nodes.
let rowid_reg = program.alloc_register();
program.emit_insn(Insn::RowId {
cursor_id,