pyturso 0.4.2__tar.gz → 0.4.3__tar.gz

{pyturso-0.4.2 → pyturso-0.4.3}/Cargo.lock

@@ -896,7 +896,7 @@ checksum = "773648b94d0e5d620f64f280777445740e61fe701025087ec8b57f45c791888b"
 
 [[package]]
 name = "core_tester"
-version = "0.4.2"
+version = "0.4.3"
 dependencies = [
  "anyhow",
  "assert_cmd",
@@ -2729,7 +2729,7 @@ dependencies = [
 
 [[package]]
 name = "limbo_completion"
-version = "0.4.2"
+version = "0.4.3"
 dependencies = [
  "mimalloc",
  "turso_ext",
@@ -2737,7 +2737,7 @@ dependencies = [
 
 [[package]]
 name = "limbo_crypto"
-version = "0.4.2"
+version = "0.4.3"
 dependencies = [
  "blake3",
  "data-encoding",
@@ -2750,7 +2750,7 @@ dependencies = [
 
 [[package]]
 name = "limbo_csv"
-version = "0.4.2"
+version = "0.4.3"
 dependencies = [
  "csv",
  "mimalloc",
@@ -2760,7 +2760,7 @@ dependencies = [
 
 [[package]]
 name = "limbo_fuzzy"
-version = "0.4.2"
+version = "0.4.3"
 dependencies = [
  "mimalloc",
  "turso_ext",
@@ -2768,7 +2768,7 @@ dependencies = [
 
 [[package]]
 name = "limbo_ipaddr"
-version = "0.4.2"
+version = "0.4.3"
 dependencies = [
  "ipnetwork",
  "mimalloc",
@@ -2777,7 +2777,7 @@ dependencies = [
 
 [[package]]
 name = "limbo_percentile"
-version = "0.4.2"
+version = "0.4.3"
 dependencies = [
  "mimalloc",
  "turso_ext",
@@ -2785,7 +2785,7 @@ dependencies = [
 
 [[package]]
 name = "limbo_regexp"
-version = "0.4.2"
+version = "0.4.3"
 dependencies = [
  "mimalloc",
  "regex",
@@ -2794,7 +2794,7 @@ dependencies = [
 
 [[package]]
 name = "limbo_sim"
-version = "0.4.2"
+version = "0.4.3"
 dependencies = [
  "anyhow",
  "bitflags 2.9.4",
@@ -2831,7 +2831,7 @@ dependencies = [
 
 [[package]]
 name = "limbo_sqlite_test_ext"
-version = "0.4.2"
+version = "0.4.3"
 dependencies = [
  "cc",
 ]
@@ -3718,7 +3718,7 @@ dependencies = [
 
 [[package]]
 name = "py-turso"
-version = "0.4.2"
+version = "0.4.3"
 dependencies = [
  "anyhow",
  "pyo3",
@@ -4595,7 +4595,7 @@ checksum = "d372029cb5195f9ab4e4b9aef550787dce78b124fcaee8d82519925defcd6f0d"
 
 [[package]]
 name = "sql_generation"
-version = "0.4.2"
+version = "0.4.3"
 dependencies = [
  "anarchist-readable-name-generator-lib",
  "anyhow",
@@ -5299,7 +5299,7 @@ checksum = "e421abadd41a4225275504ea4d6566923418b7f05506fbc9c0fe86ba7396114b"
 
 [[package]]
 name = "turso"
-version = "0.4.2"
+version = "0.4.3"
 dependencies = [
  "anyhow",
  "bytes",
@@ -5323,14 +5323,14 @@ dependencies = [
 
 [[package]]
 name = "turso-dotnet"
-version = "0.4.2"
+version = "0.4.3"
 dependencies = [
  "turso_core",
 ]
 
 [[package]]
 name = "turso-java"
-version = "0.4.2"
+version = "0.4.3"
 dependencies = [
  "jni",
  "thiserror 2.0.16",
@@ -5339,7 +5339,7 @@ dependencies = [
 
 [[package]]
 name = "turso_cli"
-version = "0.4.2"
+version = "0.4.3"
 dependencies = [
  "anyhow",
  "bytes",
@@ -5380,7 +5380,7 @@ dependencies = [
 
 [[package]]
 name = "turso_core"
-version = "0.4.2"
+version = "0.4.3"
 dependencies = [
  "aegis",
  "aes",
@@ -5447,7 +5447,7 @@ dependencies = [
 
 [[package]]
 name = "turso_dart"
-version = "0.4.2"
+version = "0.4.3"
 dependencies = [
  "flutter_rust_bridge",
  "turso_core",
@@ -5455,7 +5455,7 @@ dependencies = [
 
 [[package]]
 name = "turso_ext"
-version = "0.4.2"
+version = "0.4.3"
 dependencies = [
  "chrono",
  "getrandom 0.3.2",
@@ -5464,7 +5464,7 @@ dependencies = [
 
 [[package]]
 name = "turso_ext_tests"
-version = "0.4.2"
+version = "0.4.3"
 dependencies = [
  "env_logger 0.11.7",
  "lazy_static",
@@ -5475,7 +5475,7 @@ dependencies = [
 
 [[package]]
 name = "turso_macros"
-version = "0.4.2"
+version = "0.4.3"
 dependencies = [
  "proc-macro2",
  "quote",
@@ -5484,7 +5484,7 @@ dependencies = [
 
 [[package]]
 name = "turso_node"
-version = "0.4.2"
+version = "0.4.3"
 dependencies = [
  "chrono",
  "napi",
@@ -5497,7 +5497,7 @@ dependencies = [
 
 [[package]]
 name = "turso_parser"
-version = "0.4.2"
+version = "0.4.3"
 dependencies = [
  "bitflags 2.9.4",
  "criterion",
@@ -5514,7 +5514,7 @@ dependencies = [
 
 [[package]]
 name = "turso_sdk_kit"
-version = "0.4.2"
+version = "0.4.3"
 dependencies = [
  "bindgen",
  "env_logger 0.11.7",
@@ -5527,7 +5527,7 @@ dependencies = [
 
 [[package]]
 name = "turso_sdk_kit_macros"
-version = "0.4.2"
+version = "0.4.3"
 dependencies = [
  "proc-macro2",
  "quote",
@@ -5536,7 +5536,7 @@ dependencies = [
 
 [[package]]
 name = "turso_sqlite3"
-version = "0.4.2"
+version = "0.4.3"
 dependencies = [
  "env_logger 0.11.7",
  "libc",
@@ -5549,7 +5549,7 @@ dependencies = [
 
 [[package]]
 name = "turso_stress"
-version = "0.4.2"
+version = "0.4.3"
 dependencies = [
  "antithesis_sdk",
  "clap",
@@ -5565,7 +5565,7 @@ dependencies = [
 
 [[package]]
 name = "turso_sync_engine"
-version = "0.4.2"
+version = "0.4.3"
 dependencies = [
  "base64 0.22.1",
  "bytes",
@@ -5592,7 +5592,7 @@ dependencies = [
 
 [[package]]
 name = "turso_sync_js"
-version = "0.4.2"
+version = "0.4.3"
 dependencies = [
  "genawaiter",
  "napi",
@@ -5607,7 +5607,7 @@ dependencies = [
 
 [[package]]
 name = "turso_sync_sdk_kit"
-version = "0.4.2"
+version = "0.4.3"
 dependencies = [
  "bindgen",
  "env_logger 0.11.7",
@@ -5624,7 +5624,7 @@ dependencies = [
 
 [[package]]
 name = "turso_whopper"
-version = "0.4.2"
+version = "0.4.3"
 dependencies = [
  "anyhow",
  "clap",

{pyturso-0.4.2 → pyturso-0.4.3}/Cargo.toml

@@ -6,24 +6,24 @@ members = ["bindings/python"]
 exclude = ["perf/latency/limbo", "turso-test-runner"]
 
 [workspace.package]
-version = "0.4.2"
+version = "0.4.3"
 authors = ["the Limbo authors"]
 edition = "2021"
 license = "MIT"
 repository = "https://github.com/tursodatabase/turso"
 
 [workspace.dependencies]
-turso = { path = "bindings/rust", version = "0.4.2" }
-turso_node = { path = "bindings/javascript", version = "0.4.2" }
-turso_sdk_kit = { path = "sdk-kit", version = "0.4.2" }
-turso_sdk_kit_macros = { path = "sdk-kit-macros", version = "0.4.2" }
-turso_sync_sdk_kit = { path = "sync/sdk-kit", version = "0.4.2" }
-limbo_completion = { path = "extensions/completion", version = "0.4.2" }
-turso_core = { path = "core", version = "0.4.2" }
-turso_sync_engine = { path = "sync/engine", version = "0.4.2" }
-turso_ext = { path = "extensions/core", version = "0.4.2" }
-turso_macros = { path = "macros", version = "0.4.2" }
-turso_parser = { path = "parser", version = "0.4.2" }
+turso = { path = "bindings/rust", version = "0.4.3" }
+turso_node = { path = "bindings/javascript", version = "0.4.3" }
+turso_sdk_kit = { path = "sdk-kit", version = "0.4.3" }
+turso_sdk_kit_macros = { path = "sdk-kit-macros", version = "0.4.3" }
+turso_sync_sdk_kit = { path = "sync/sdk-kit", version = "0.4.3" }
+limbo_completion = { path = "extensions/completion", version = "0.4.3" }
+turso_core = { path = "core", version = "0.4.3" }
+turso_sync_engine = { path = "sync/engine", version = "0.4.3" }
+turso_ext = { path = "extensions/core", version = "0.4.3" }
+turso_macros = { path = "macros", version = "0.4.3" }
+turso_parser = { path = "parser", version = "0.4.3" }
 sql_generation = { path = "sql_generation" }
 strum = { version = "0.26", features = ["derive"] }
 strum_macros = "0.26"

{pyturso-0.4.2 → pyturso-0.4.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pyturso
-Version: 0.4.2
+Version: 0.4.3
 Classifier: Development Status :: 3 - Alpha
 Classifier: Programming Language :: Python
 Classifier: Programming Language :: Python :: 3

{pyturso-0.4.2 → pyturso-0.4.3}/core/function.rs

@@ -611,7 +611,7 @@ impl Display for MathFunc {
     }
 }
 
-#[derive(Debug)]
+#[derive(Debug, Clone)]
 pub enum AlterTableFunc {
     RenameTable,
     AlterColumn,
@@ -628,7 +628,7 @@ impl Display for AlterTableFunc {
     }
 }
 
-#[derive(Debug)]
+#[derive(Debug, Clone)]
 pub enum Func {
     Agg(AggFunc),
     Scalar(ScalarFunc),
@@ -655,7 +655,7 @@ impl Display for Func {
     }
 }
 
-#[derive(Debug)]
+#[derive(Debug, Clone)]
 pub struct FuncCtx {
     pub func: Func,
     pub arg_count: usize,

{pyturso-0.4.2 → pyturso-0.4.3}/core/lib.rs

@@ -110,7 +110,7 @@ use util::parse_schema_rows;
 pub use util::IOExt;
 pub use vdbe::{
     builder::QueryMode, explain::EXPLAIN_COLUMNS, explain::EXPLAIN_QUERY_PLAN_COLUMNS,
-    FromValueRow, Register,
+    FromValueRow, Program, Register,
 };
 
 /// Configuration for database features
@@ -682,9 +682,11 @@ impl Database {
         // MVCC currently requires a WAL open to function
         let shared_wal = WalFileShared::open_shared_if_exists(&self.io, &self.wal_path, flags)?;
 
+        let last_checksum_and_max_frame = shared_wal.read().last_checksum_and_max_frame();
         let wal = Arc::new(WalFile::new(
             self.io.clone(),
             Arc::clone(&shared_wal),
+            last_checksum_and_max_frame,
             pager.buffer_pool.clone(),
         ));
 
@@ -894,6 +896,7 @@ impl Database {
             Some(Arc::new(WalFile::new(
                 self.io.clone(),
                 self.shared_wal.clone(),
+                shared_wal.last_checksum_and_max_frame(),
                 buffer_pool.clone(),
             )))
         } else {

{pyturso-0.4.2 → pyturso-0.4.3}/core/parameters.rs

@@ -21,7 +21,7 @@ impl Parameter {
     }
 }
 
-#[derive(Debug)]
+#[derive(Debug, Clone)]
 pub struct Parameters {
     next_index: NonZero<usize>,
     pub list: Vec<Parameter>,

{pyturso-0.4.2 → pyturso-0.4.3}/core/statement.rs

@@ -86,8 +86,16 @@ impl Statement {
         self.query_mode
     }
 
+    pub fn get_program(&self) -> &vdbe::Program {
+        &self.program
+    }
+
+    pub fn get_pager(&self) -> &Arc<Pager> {
+        &self.pager
+    }
+
     pub fn n_change(&self) -> i64 {
-        self.program
+        self.state
             .n_change
             .load(std::sync::atomic::Ordering::SeqCst)
     }
@@ -451,7 +459,7 @@ impl Statement {
         // as abort uses auto_txn_cleanup value - it needs to be called before state.reset
         self.program.abort(&self.pager, None, &mut self.state);
         self.state.reset(max_registers, max_cursors);
-        self.program.n_change.store(0, Ordering::SeqCst);
+        self.state.n_change.store(0, Ordering::SeqCst);
         self.busy = false;
         self.busy_handler_state = None;
     }

{pyturso-0.4.2 → pyturso-0.4.3}/core/storage/btree.rs

@@ -9097,7 +9097,13 @@ mod tests {
 
         let wal_file = io.open_file("test.wal", OpenFlags::Create, false).unwrap();
         let wal_shared = WalFileShared::new_shared(wal_file).unwrap();
-        let wal: Arc<dyn Wal> = Arc::new(WalFile::new(io.clone(), wal_shared, buffer_pool.clone()));
+        let last_checksum_and_max_frame = wal_shared.read().last_checksum_and_max_frame();
+        let wal: Arc<dyn Wal> = Arc::new(WalFile::new(
+            io.clone(),
+            wal_shared,
+            last_checksum_and_max_frame,
+            buffer_pool.clone(),
+        ));
 
         // For new empty databases, init_page_1 must be Some(page) so allocate_page1() can be called
         let init_page_1 = Arc::new(ArcSwapOption::new(Some(default_page1(None))));

{pyturso-0.4.2 → pyturso-0.4.3}/core/storage/pager.rs

@@ -4139,8 +4139,17 @@ impl Pager {
     #[instrument(skip_all, level = Level::DEBUG)]
     pub fn rollback(&self, schema_did_change: bool, connection: &Connection, is_write: bool) {
         tracing::debug!(schema_did_change);
-        self.clear_page_cache(is_write);
         if is_write {
+            let clear_dirty = true;
+            // The page cache only needs to be cleared if we are rolling back a write transaction.
+            // If a read transaction rolls back, and the next read transaction detects that the
+            // database has changed in between (see db_changed() in wal.rs), then the page cache
+            // will be cleared. Since the read transaction itself has not modified anything, it can proceed
+            // with its cached pages in case the database has NOT changed in between.
+            //
+            // Even in the case of a write transaction, clearing the entire page cache is overkill,
+            // since we only need to clear the dirty pages that were modified by the write transaction.
+            self.clear_page_cache(clear_dirty);
             self.dirty_pages.write().clear();
         } else {
             turso_assert!(
@@ -4296,7 +4305,7 @@ pub fn default_page1(cipher: Option<&CipherMode>) -> PageRef {
     page
 }
 
-#[derive(Debug)]
+#[derive(Debug, Clone, Copy)]
 pub struct CreateBTreeFlags(pub u8);
 impl CreateBTreeFlags {
     pub const TABLE: u8 = 0b0001;
@@ -4572,13 +4581,16 @@ mod ptrmap_tests {
         let buffer_pool = BufferPool::begin_init(&io, (sz * page_size) as usize);
         let page_cache = Arc::new(RwLock::new(PageCache::new(sz as usize)));
 
+        let wal_shared = WalFileShared::new_shared(
+            io.open_file("test.db-wal", OpenFlags::Create, false)
+                .unwrap(),
+        )
+        .unwrap();
+        let last_checksum_and_max_frame = wal_shared.read().last_checksum_and_max_frame();
         let wal: Arc<dyn Wal> = Arc::new(WalFile::new(
             io.clone(),
-            WalFileShared::new_shared(
-                io.open_file("test.db-wal", OpenFlags::Create, false)
-                    .unwrap(),
-            )
-            .unwrap(),
+            wal_shared,
+            last_checksum_and_max_frame,
             buffer_pool.clone(),
         ));
 

{pyturso-0.4.2 → pyturso-0.4.3}/core/storage/wal.rs

@@ -1924,15 +1924,9 @@ impl WalFile {
     pub fn new(
         io: Arc<dyn IO>,
         shared: Arc<RwLock<WalFileShared>>,
+        (last_checksum, max_frame): ((u32, u32), u64),
         buffer_pool: Arc<BufferPool>,
     ) -> Self {
-        let (last_checksum, max_frame) = {
-            let shared_guard = shared.read();
-            (
-                shared_guard.last_checksum,
-                shared_guard.max_frame.load(Ordering::Acquire),
-            )
-        };
         let now = io.now();
         Self {
             io,
@@ -2693,6 +2687,9 @@ impl WalFile {
 }
 
 impl WalFileShared {
+    pub fn last_checksum_and_max_frame(&self) -> ((u32, u32), u64) {
+        (self.last_checksum, self.max_frame.load(Ordering::Acquire))
+    }
     pub fn open_shared_if_exists(
         io: &Arc<dyn IO>,
         path: &str,

{pyturso-0.4.2 → pyturso-0.4.3}/core/translate/schema.rs

@@ -29,6 +29,9 @@ fn validate(body: &ast::CreateTableBody, connection: &Connection) -> Result<()>
         options, columns, ..
     } = &body
     {
+        if options.contains(ast::TableOptions::WITHOUT_ROWID) {
+            bail_parse_error!("WITHOUT ROWID tables are not supported");
+        }
         if options.contains(ast::TableOptions::STRICT) && !connection.experimental_strict_enabled()
         {
             bail_parse_error!(

{pyturso-0.4.2 → pyturso-0.4.3}/core/vdbe/builder.rs

@@ -1,7 +1,6 @@
-use parking_lot::RwLock;
 use std::{
     collections::HashMap,
-    sync::{atomic::AtomicI64, Arc},
+    sync::Arc,
 };
 
 use tracing::{instrument, Level};
@@ -39,7 +38,7 @@ impl TableRefIdCounter {
     }
 }
 
-use super::{BranchOffset, CursorID, ExplainState, Insn, InsnReference, JumpTarget, Program};
+use super::{BranchOffset, CursorID, Insn, InsnReference, JumpTarget, Program};
 
 /// A key that uniquely identifies a cursor.
 /// The key is a pair of table reference id and index.
@@ -1264,7 +1263,6 @@ impl ProgramBuilder {
             comments: self.comments,
             connection,
             parameters: self.parameters,
-            n_change: AtomicI64::new(0),
             change_cnt_on,
             result_columns: self.result_columns,
             table_references: self.table_references,
@@ -1275,7 +1273,6 @@ impl ProgramBuilder {
             is_subprogram: self.is_subprogram,
             contains_trigger_subprograms,
             resolve_type: self.resolve_type,
-            explain_state: RwLock::new(ExplainState::default()),
         })
     }
 }

{pyturso-0.4.2 → pyturso-0.4.3}/core/vdbe/execute.rs

@@ -6440,7 +6440,7 @@ pub fn op_insert(
                 };
                 if let Some(rowid) = maybe_rowid {
                     program.connection.update_last_rowid(rowid);
-                    program
+                    state
                         .n_change
                         .fetch_add(1, std::sync::atomic::Ordering::SeqCst);
                 }
@@ -6660,7 +6660,7 @@ pub fn op_delete(
     if !is_part_of_update {
         // DELETEs do not count towards the total changes if they are part of an UPDATE statement,
         // i.e. the DELETE and subsequent INSERT of a row are the same "change".
-        program
+        state
             .n_change
             .fetch_add(1, std::sync::atomic::Ordering::SeqCst);
     }

{pyturso-0.4.2 → pyturso-0.4.3}/core/vdbe/insn.rs

@@ -169,7 +169,7 @@ impl<T: Copy + std::fmt::Display> std::fmt::Display for RegisterOrLiteral<T> {
 }
 
 /// Data for HashBuild instruction (boxed to keep Insn small).
-#[derive(Debug)]
+#[derive(Debug, Clone)]
 pub struct HashBuildData {
     pub cursor_id: CursorID,
     pub key_start_reg: usize,
@@ -186,7 +186,7 @@ pub struct HashBuildData {
 
 // There are currently 190 opcodes in sqlite
 #[repr(u8)]
-#[derive(Description, Debug, EnumDiscriminants)]
+#[derive(Description, Debug, Clone, EnumDiscriminants)]
 #[strum_discriminants(vis(pub(crate)))]
 #[strum_discriminants(derive(VariantArray, EnumCount, FromRepr))]
 #[strum_discriminants(name(InsnVariants))]