This repository was archived by the owner on Jan 6, 2026. It is now read-only.
Commit c5e55b3
authored
Merge 'Consume all rows' from Nikita Sivukhin
This PR upgrade turso_core to version `0.3.2`. This lead to regression,
so few places also fixed in order to mitigate new issues.
The main issue is that closing rows without error can silently reset all
changes made within a statement.
Go bindings for now can hit this behavior in case of `INSERT` with
`RETURNING` clause as only one row will be consumed before clause for
them (but final Done will not be reached)
This PR explicitly consume all rows on Close if there were no error for
the statement
Reviewed-by: Preston Thorpe <preston@turso.tech>
Closes #6220 files changed
Lines changed: 76 additions & 18 deletions
File tree
- libs
- darwin_amd64
- darwin_arm64
- linux_amd64
- linux_arm64
- linux_musl_amd64
- linux_musl_arm64
- windows_amd64
- rs_src
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
15 | 15 | | |
16 | 16 | | |
17 | 17 | | |
18 | | - | |
| 18 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
447 | 447 | | |
448 | 448 | | |
449 | 449 | | |
| 450 | + | |
| 451 | + | |
| 452 | + | |
| 453 | + | |
| 454 | + | |
450 | 455 | | |
451 | 456 | | |
452 | 457 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | | - | |
2 | | - | |
3 | | - | |
4 | | - | |
5 | | - | |
6 | | - | |
7 | | - | |
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
Binary file not shown.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | | - | |
| 1 | + | |
Binary file not shown.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | | - | |
| 1 | + | |
0 commit comments