diff options
author | Julien Kritter <julien.kritter@withings.com> | 2024-09-12 10:26:28 +0200 |
---|---|---|
committer | Julien Kritter <julien.kritter@withings.com> | 2024-09-12 10:26:28 +0200 |
commit | 51ced6036602826ff653c6fc283cbf11fc7b4865 (patch) | |
tree | d381b99333b55f58e5a05cbb197712ffd4ab1a43 /src/db/lmdb_adapter.rs | |
parent | 586957b4b7261b43e7820d7193516153201b4954 (diff) | |
download | garage-51ced6036602826ff653c6fc283cbf11fc7b4865.tar.gz garage-51ced6036602826ff653c6fc283cbf11fc7b4865.zip |
Don't fetch old values in cross-partition transactional inserts
Diffstat (limited to 'src/db/lmdb_adapter.rs')
-rw-r--r-- | src/db/lmdb_adapter.rs | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/src/db/lmdb_adapter.rs b/src/db/lmdb_adapter.rs index 436a67fa..de4c3910 100644 --- a/src/db/lmdb_adapter.rs +++ b/src/db/lmdb_adapter.rs @@ -252,17 +252,15 @@ impl<'a> ITx for LmdbTx<'a> { Ok(tree.len(&self.tx)? as usize) } - fn insert(&mut self, tree: usize, key: &[u8], value: &[u8]) -> TxOpResult<Option<Value>> { + fn insert(&mut self, tree: usize, key: &[u8], value: &[u8]) -> TxOpResult<()> { let tree = *self.get_tree(tree)?; - let old_val = tree.get(&self.tx, key)?.map(Vec::from); tree.put(&mut self.tx, key, value)?; - Ok(old_val) + Ok(()) } - fn remove(&mut self, tree: usize, key: &[u8]) -> TxOpResult<Option<Value>> { + fn remove(&mut self, tree: usize, key: &[u8]) -> TxOpResult<()> { let tree = *self.get_tree(tree)?; - let old_val = tree.get(&self.tx, key)?.map(Vec::from); tree.delete(&mut self.tx, key)?; - Ok(old_val) + Ok(()) } fn clear(&mut self, tree: usize) -> TxOpResult<()> { let tree = *self.get_tree(tree)?; |