convert_db: prevent conversion between same input/output engine

Use optional DB open overrides for both input and output database.

Duplicating the same override flag for input/output would result in too
many, too long flags. It would be too costly for very rare edge-case
where converting between same DB engine, just with different flags.

Because overrides flags for different engines are disjoint and we are
preventing conversion between same input/ouput DB engine, we can have
only one set.

The override flag will be passed either to input or output, based on
engine type it belongs to. It will never be passed to both of them and
cause unwelcome surprise to user.

1 files changed, 47 insertions, 0 deletions

diff --git a/src/db/lib.rs b/src/db/lib.rs
index fe44b01e..427140f9 100644
--- a/src/db/lib.rs
+++ b/src/db/lib.rs
@@ -171,6 +171,53 @@ impl Db {
 	}
 }
 
+/// List of supported database engine types
+#[derive(Clone, Copy, Debug, PartialEq, Eq)]
+pub enum Engine {
+	#[cfg(feature = "lmdb")]
+	Lmdb,
+
+	#[cfg(feature = "sqlite")]
+	Sqlite,
+
+	#[cfg(feature = "sled")]
+	Sled,
+}
+
+impl std::fmt::Display for Engine {
+	fn fmt(&self, fmt: &mut std::fmt::Formatter) -> std::fmt::Result {
+		match self {
+			#[cfg(feature = "lmdb")]
+			Self::Lmdb => fmt.write_str("lmdb"),
+
+			#[cfg(feature = "sqlite")]
+			Self::Sqlite => fmt.write_str("sqlite"),
+
+			#[cfg(feature = "sled")]
+			Self::Sled => fmt.write_str("sled"),
+		}
+	}
+}
+
+impl std::str::FromStr for Engine {
+	type Err = Error;
+
+	fn from_str(text: &str) -> Result<Engine> {
+		match text {
+			#[cfg(feature = "lmdb")]
+			"lmdb" | "heed" => Ok(Self::Lmdb),
+
+			#[cfg(feature = "sqlite")]
+			"sqlite" | "sqlite3" | "rusqlite" => Ok(Self::Sqlite),
+
+			#[cfg(feature = "sled")]
+			"sled" => Ok(Self::Sled),
+
+			kind => Err(Error(format!("Invalid DB engine: {}", kind).into())),
+		}
+	}
+}
+
 #[allow(clippy::len_without_is_empty)]
 impl Tree {
 	#[inline]