Merge pull request #907 from spacejam/tyler_0.30

0.30

CHANGELOG.md

@@ -1,6 +1,13 @@
-# Upcoming
+# 0.30
 
-* Migrated to a new serialization format
+## Breaking Changes
+
+* Migrated to a new storage format
+
+## Bugfixes
+
+* Fixed a bug where cache was not being evicted
+* Fixed a bug with using transactions with compression
 
 # 0.29.2
 

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "sled"
-version = "0.29.2"
+version = "0.30.0"
 authors = ["Tyler Neely <[email protected]>"]
 description = "a modern embedded database"
 license = "MIT/Apache-2.0"
@@ -11,6 +11,7 @@ categories = ["database-implementations", "concurrency", "data-structures", "alg
 documentation = "https://docs.rs/sled/"
 readme = "README.md"
 edition = "2018"
+exclude = ["benchmarks", "examples", "bindings", "scripts"]
 
 [profile.release]
 debug = true

src/pagecache/logger.rs

@@ -134,7 +134,7 @@ impl Log {
 
         #[cfg(feature = "compression")]
         {
-            if self.config.use_compression {
+            if self.config.use_compression && pid != BATCH_MANIFEST_PID {
                 use zstd::block::compress;
 
                 let _measure = Measure::new(&M.compress);

src/pagecache/reservation.rs

@@ -91,7 +91,7 @@ impl<'a> Reservation<'a> {
 
         self.buf[0] = MessageKind::BatchManifest.into();
 
-        let buf = u64_to_arr(u64::try_from(lsn).unwrap());
+        let buf = lsn_to_arr(lsn);
 
         let dst = &mut self.buf[MSG_HEADER_LEN..];
 

tests/test_tree.rs

@@ -329,7 +329,10 @@ fn concurrent_tree_iter() -> Result<()> {
 fn concurrent_tree_transactions() -> TransactionResult<()> {
     common::setup_logger();
 
-    let config = Config::new().temporary(true).flush_every_ms(None);
+    let config = Config::new()
+        .temporary(true)
+        .flush_every_ms(None)
+        .use_compression(true);
     let db = config.open().unwrap();
 
     db.insert(b"k1", b"cats").unwrap();