Make encode/decode bounds non-default.

Author: gz

.github/workflows/main.yml

@@ -267,7 +267,7 @@ jobs:
         uses: actions-rs/cargo@v1
         with:
           command: rustdoc
-          args: --all-features -- -D warnings --cfg docsrs
+          args: ${{ env.ALMOST_ALL_FEATURES }} -- -D warnings --cfg docsrs
 
   udeps:
     name: Unused Dependencies
@@ -297,4 +297,4 @@ jobs:
         uses: actions-rs/cargo@v1
         with:
           command: udeps
-          args: --all --all-features --all-targets
+          args: --all ${{ env.ALMOST_ALL_FEATURES }} --all-targets

Cargo.toml

@@ -12,7 +12,7 @@ rustdoc-args = ["--cfg", "docsrs"]
 # Note: If you add a feature, adjust the ALMOST_ALL_FEATURES environment variable in
 # main.yml and coverage.yml:
 default = ["with-serde"]
-persistence = []
+persistence = ["rocksdb", "uuid"]
 with-serde = ["serde"]
 with-csv = ["csv"]
 with-nexmark = [
@@ -52,9 +52,9 @@ paste = { version = "1.0.9", optional = true }
 bitvec = "1.0.1"
 xxhash-rust = { version = "0.8.6", features = ["xxh3"] }
 crossbeam = "0.8.2"
-rocksdb = { version = "0.19", default-features = false, features = ["multi-threaded-cf"] }
+rocksdb = { version = "0.19", default-features = false, features = ["multi-threaded-cf"], optional = true }
 bincode = "2.0.0-rc.2"
-uuid = { version = "1.1.2",  features = ["v4"] }
+uuid = { version = "1.1.2",  features = ["v4"], optional = true }
 
 # TODO: Remove these dependencies
 rand = { version = "0.8", optional = true }

src/trace/mod.rs

@@ -14,6 +14,7 @@ pub mod consolidation;
 pub mod cursor;
 pub mod layers;
 pub mod ord;
+#[cfg(feature = "persistence")]
 pub mod persistent;
 // We export the `spine_fueled` module only for testing (so we can explicitly
 // choose the in-memory DS). For regular logic, since we replace it with the
@@ -30,6 +31,7 @@ use crate::{
     time::{AntichainRef, Timestamp},
     NumEntries,
 };
+#[cfg(feature = "persistence")]
 use bincode::{Decode, Encode};
 pub use cursor::{Consumer, Cursor, ValueConsumer};
 #[cfg(feature = "persistence")]
@@ -46,15 +48,30 @@ use std::{fmt::Debug, hash::Hash};
 /// must be generic over any relational data, it is sufficient to impose
 /// `DBData` as a trait bound on types.  Conversely, a trait bound of the form
 /// `B: BatchReader` implies `B::Key: DBData` and `B::Val: DBData`.
+#[cfg(feature = "persistence")]
 pub trait DBData:
     Clone + Eq + Ord + Hash + SizeOf + Send + Debug + Decode + Encode + 'static
 {
 }
+
+#[cfg(not(feature = "persistence"))]
+pub trait DBData:
+    Clone + Eq + Ord + Hash + SizeOf + Send + Debug + 'static
+{
+}
+
+#[cfg(feature = "persistence")]
 impl<T> DBData for T where
     T: Clone + Eq + Ord + Hash + SizeOf + Send + Debug + Decode + Encode + 'static
 {
 }
 
+#[cfg(not(feature = "persistence"))]
+impl<T> DBData for T where
+    T: Clone + Eq + Ord + Hash + SizeOf + Send + Debug + 'static
+{
+}
+
 /// Trait for data types used as weights.
 ///
 /// A type used for weights in a batch (i.e., as `BatchReader::R`) must behave