Merge 'translate: disable support for UPDATE ... ORDER BY' from Jussi Saurio

we haven't really tested this at all, and it clearly doesn't work, as
per #3315.
best to disable it for the time being, especially since vanilla SQLite
doesn't support this syntax unless you compile it with a flag.

Closes #3337
This commit is contained in:
Pekka Enberg
2025-09-25 17:24:15 +03:00
committed by GitHub

View File

@@ -122,6 +122,11 @@ pub fn prepare_update_plan(
{
bail_parse_error!("INDEXED BY clause is not supported in UPDATE");
}
if !body.order_by.is_empty() {
bail_parse_error!("ORDER BY is not supported in UPDATE");
}
let table_name = &body.tbl_name.name;
// Check if this is a system table that should be protected from direct writes