deprecate GrowingStore and RevertibleStore

basecoin/store/src/impls/growing.rs

@@ -8,6 +8,7 @@ use crate::types::{Height, Path};
 /// If the path is deleted, the stored value is []
 /// Note: we should not allow empty vec to store as
 /// this would conflict with the deletion representation.
+#[deprecated(since = "TBD", note = "InMemoryStore implements deletion in AVL tree.")]
 #[derive(Clone, Debug)]
 pub struct GrowingStore<S> {
     store: S,

basecoin/store/src/impls/revertible.rs

@@ -16,6 +16,10 @@ use crate::types::{Height, Path};
 /// an overwriting `set` doesn't reorganize a Merkle tree - but non-overwriting `set` and `delete`
 /// operations may reorganize a Merkle tree - which may change the root hash. However, a Merkle
 /// store should have no effect on a failed transaction.
+#[deprecated(
+    since = "TBD",
+    note = "Using operation log to revert changes does not guarantee deterministic Merkle root hash."
+)]
 #[derive(Clone, Debug)]
 pub struct RevertibleStore<S> {
     /// backing store
@@ -95,6 +99,12 @@ where
         Ok(())
     }
 
+    /// Revert all operations in the operation log.
+    ///
+    /// This method doesn't guarantee the Merkle tree with the exactly previous root hash.
+    /// It should be avoided. Use `InMemoryStore` directly which implements rollback directly.
+    ///
+    /// GH issue: informalsystems/basecoin-rs#129
     #[inline]
     fn reset(&mut self) {
         // note that we do NOT call the backing store's reset here - this allows users to create