From fb2878973f6237549f3dbed9c8b33aee0a672d7a Mon Sep 17 00:00:00 2001
From: Jussi Saurio <jussi.saurio@gmail.com>
Date: Tue, 30 Sep 2025 22:54:36 +0300
Subject: [PATCH] fix sort order of write set

---
 core/mvcc/database/mod.rs | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/core/mvcc/database/mod.rs b/core/mvcc/database/mod.rs
index a11689e27..31bc2c199 100644
--- a/core/mvcc/database/mod.rs
+++ b/core/mvcc/database/mod.rs
@@ -527,8 +527,10 @@ impl<Clock: LogicalClock> StateTransition for CommitStateMachine<Clock> {
                 tracing::trace!("commit_tx(tx_id={})", self.tx_id);
                 self.write_set
                     .extend(tx.write_set.iter().map(|v| *v.value()));
-                self.write_set
-                    .sort_by(|a, b| a.table_id.cmp(&b.table_id).then(a.row_id.cmp(&b.row_id)));
+                self.write_set.sort_by(|a, b| {
+                    // table ids are negative, and sqlite_schema has id -1 so we want to sort in descending order of table id
+                    b.table_id.cmp(&a.table_id).then(a.row_id.cmp(&b.row_id))
+                });
                 if self.write_set.is_empty() {
                     tx.state.store(TransactionState::Committed(end_ts));
                     if mvcc_store.is_exclusive_tx(&self.tx_id) {