Added the actual implementation for the AtomicView::latest_view

CHANGELOG.md

@@ -7,6 +7,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 ## [Unreleased]
 
 ### Added
+- [#1994](https://github.com/FuelLabs/fuel-core/pull/1994): Added the actual implementation for the `AtomicView::latest_view`.
 - [#1972](https://github.com/FuelLabs/fuel-core/pull/1972): Implement `AlgorithmUpdater` for `GasPriceService`
 - [#1948](https://github.com/FuelLabs/fuel-core/pull/1948): Add new `AlgorithmV1` and `AlgorithmUpdaterV1` for the gas price. Include tools for analysis
 

crates/fuel-core/src/database.rs

@@ -19,7 +19,6 @@ use crate::{
         },
         generic_database::GenericDatabase,
         in_memory::memory_store::MemoryStore,
-        iterable_key_value_view::IterableKeyValueViewWrapper,
         key_value_view::KeyValueViewWrapper,
         ChangesIterator,
         ColumnType,
@@ -249,10 +248,7 @@ where
     type LatestView = IterableKeyValueView<ColumnType<Description>>;
 
     fn latest_view(&self) -> StorageResult<Self::LatestView> {
-        // TODO: https://github.com/FuelLabs/fuel-core/issues/1581
-        Ok(IterableKeyValueView::from_storage(
-            IterableKeyValueViewWrapper::new(self.clone()),
-        ))
+        self.inner_storage().data.latest_view()
     }
 }
 

crates/fuel-core/src/schema.rs

@@ -213,9 +213,9 @@ impl<'a> ReadViewProvider for Context<'a> {
     fn read_view(&self) -> StorageResult<Cow<'a, ReadView>> {
         let operation_type = self.query_env.operation.node.ty;
 
-        // Sometimes, during mutable queries the resolvers
+        // Sometimes, during mutable queries or subscription the resolvers
         // need access to an updated view of the database.
-        if operation_type == OperationType::Mutation {
+        if operation_type != OperationType::Query {
             let database: &ReadDatabase = self.data_unchecked();
             database.view().map(Cow::Owned)
         } else {

crates/fuel-core/src/schema/block.rs

@@ -374,7 +374,6 @@ impl BlockMutation {
         start_timestamp: Option<Tai64Timestamp>,
         blocks_to_produce: U32,
     ) -> async_graphql::Result<U32> {
-        let query = ctx.read_view()?;
         let consensus_module = ctx.data_unchecked::<ConsensusModule>();
         let config = ctx.data_unchecked::<GraphQLConfig>().clone();
 
@@ -388,7 +387,7 @@ impl BlockMutation {
             .manually_produce_blocks(start_time, blocks_to_produce)
             .await?;
 
-        query
+        ctx.read_view()?
             .latest_block_height()
             .map(Into::into)
             .map_err(Into::into)

crates/fuel-core/src/service/genesis/importer/import_task.rs

@@ -205,6 +205,7 @@ mod tests {
         },
         state::{
             in_memory::memory_store::MemoryStore,
+            IterableKeyValueView,
             TransactableStorage,
         },
     };
@@ -561,6 +562,10 @@ mod tests {
         ) -> StorageResult<()> {
             Err(anyhow::anyhow!("I refuse to work!").into())
         }
+
+        fn latest_view(&self) -> StorageResult<IterableKeyValueView<Self::Column>> {
+            Err(anyhow::anyhow!("I refuse to work!").into())
+        }
     }
 
     #[test]

crates/fuel-core/src/state.rs

@@ -61,6 +61,8 @@ pub trait TransactableStorage<Height>: IterableStore + Debug + Send + Sync {
         height: Option<Height>,
         changes: Changes,
     ) -> StorageResult<()>;
+
+    fn latest_view(&self) -> StorageResult<IterableKeyValueView<Self::Column>>;
 }
 
 // It is used only to allow conversion of the `StorageTransaction` into the `DataSource`.
@@ -73,6 +75,10 @@ where
     fn commit_changes(&self, _: Option<Height>, _: Changes) -> StorageResult<()> {
         unimplemented!()
     }
+
+    fn latest_view(&self) -> StorageResult<IterableKeyValueView<Self::Column>> {
+        unimplemented!()
+    }
 }
 
 /// A type that allows to iterate over the `Changes`.

crates/fuel-core/src/state/generic_database.rs

@@ -28,6 +28,10 @@ pub struct GenericDatabase<Storage> {
 }
 
 impl<Storage> GenericDatabase<Storage> {
+    pub fn inner_storage(&self) -> &Storage {
+        self.storage.as_ref()
+    }
+
     pub fn from_storage(storage: Storage) -> Self {
         Self {
             storage: StructuredStorage::new(storage),

crates/fuel-core/src/state/in_memory.rs

@@ -1,2 +1,3 @@
 // TODO: Move the implementation from the module to here.
 pub mod memory_store;
+pub mod memory_view;

crates/fuel-core/src/state/in_memory/memory_store.rs

@@ -4,7 +4,10 @@ use crate::{
         DatabaseDescription,
     },
     state::{
+        in_memory::memory_view::MemoryView,
+        iterable_key_value_view::IterableKeyValueViewWrapper,
         IterDirection,
+        IterableKeyValueView,
         TransactableStorage,
     },
 };
@@ -31,7 +34,11 @@ use fuel_core_storage::{
 use std::{
     collections::BTreeMap,
     fmt::Debug,
-    sync::Mutex,
+    ops::Deref,
+    sync::{
+        Arc,
+        Mutex,
+    },
 };
 
 #[derive(Debug)]
@@ -62,6 +69,23 @@ impl<Description> MemoryStore<Description>
 where
     Description: DatabaseDescription,
 {
+    fn create_view(&self) -> MemoryView<Description> {
+        // Lock all tables at the same time to have consistent view.
+        let locks = self
+            .inner
+            .iter()
+            .map(|lock| lock.lock().expect("Poisoned lock"))
+            .collect::<Vec<_>>();
+        let inner = locks
+            .iter()
+            .map(|btree| btree.deref().clone())
+            .collect::<Vec<_>>();
+        MemoryView {
+            inner,
+            _marker: Default::default(),
+        }
+    }
+
     pub fn iter_all(
         &self,
         column: Description::Column,
@@ -136,6 +160,13 @@ where
         }
         Ok(())
     }
+
+    fn latest_view(&self) -> StorageResult<IterableKeyValueView<Self::Column>> {
+        let view = self.create_view();
+        Ok(IterableKeyValueView::from_storage(
+            IterableKeyValueViewWrapper::new(Arc::new(view)),
+        ))
+    }
 }
 
 #[cfg(test)]

crates/fuel-core/src/state/in_memory/memory_view.rs

@@ -0,0 +1,76 @@
+use crate::database::database_description::{
+    on_chain::OnChain,
+    DatabaseDescription,
+};
+use fuel_core_storage::{
+    iter::{
+        iterator,
+        BoxedIter,
+        IntoBoxedIter,
+        IterDirection,
+        IterableStore,
+    },
+    kv_store::{
+        KVItem,
+        KeyValueInspect,
+        StorageColumn,
+        Value,
+    },
+    transactional::ReferenceBytesKey,
+    Result as StorageResult,
+};
+use std::collections::BTreeMap;
+
+#[derive(Debug, Clone)]
+pub struct MemoryView<Description = OnChain>
+where
+    Description: DatabaseDescription,
+{
+    pub(crate) inner: Vec<BTreeMap<ReferenceBytesKey, Value>>,
+    pub(crate) _marker: core::marker::PhantomData<Description>,
+}
+
+impl<Description> MemoryView<Description>
+where
+    Description: DatabaseDescription,
+{
+    pub fn iter_all<'a>(
+        &'a self,
+        column: Description::Column,
+        prefix: Option<&[u8]>,
+        start: Option<&[u8]>,
+        direction: IterDirection,
+    ) -> impl Iterator<Item = KVItem> + 'a {
+        let btree = &self.inner[column.as_usize()];
+
+        iterator(btree, prefix, start, direction)
+            .map(|(key, value)| (key.clone().into(), value.clone()))
+            .map(Ok)
+    }
+}
+
+impl<Description> KeyValueInspect for MemoryView<Description>
+where
+    Description: DatabaseDescription,
+{
+    type Column = Description::Column;
+
+    fn get(&self, key: &[u8], column: Self::Column) -> StorageResult<Option<Value>> {
+        Ok(self.inner[column.as_usize()].get(key).cloned())
+    }
+}
+
+impl<Description> IterableStore for MemoryView<Description>
+where
+    Description: DatabaseDescription,
+{
+    fn iter_store(
+        &self,
+        column: Self::Column,
+        prefix: Option<&[u8]>,
+        start: Option<&[u8]>,
+        direction: IterDirection,
+    ) -> BoxedIter<KVItem> {
+        self.iter_all(column, prefix, start, direction).into_boxed()
+    }
+}