vm: add mempool

Signed-off-by: Sam Batschelet <[email protected]>

Author: hexfusion

mini-kvvm/src/lib.rs

@@ -2,5 +2,6 @@ pub mod api;
 pub mod block;
 pub mod chain;
 pub mod genesis;
+pub mod mempool;
 pub mod utils;
 pub mod vm;

mini-kvvm/src/mempool/data.rs

@@ -0,0 +1,88 @@
+use std::{
+    collections::{HashMap, VecDeque},
+    io::Result,
+};
+
+use avalanche_types::ids;
+
+use crate::chain::tx::tx::Transaction;
+
+#[derive(Debug)]
+/// Memory representation of mempool
+pub struct Data {
+    pub items: VecDeque<Entry>,
+    pub lookup: HashMap<ids::Id, Entry>,
+}
+
+#[derive(Debug, Default, Clone)]
+pub struct Entry {
+    pub id: ids::Id,
+    pub tx: Option<Transaction>,
+    pub index: usize,
+}
+
+impl Data {
+    pub fn new(max_size: usize) -> Self {
+        Self {
+            items: VecDeque::with_capacity(max_size),
+            lookup: HashMap::new(),
+        }
+    }
+
+    pub fn len(&self) -> usize {
+        self.items.len()
+    }
+
+
+    pub fn is_empty(&self) -> bool {
+        self.items.len() == 0
+    }
+
+    pub fn less(&self, _i: usize, _j: usize) -> bool {
+        // return self.items[i].price > self.items[j].price;
+        false
+    }
+
+    pub fn swap(&mut self, i: usize, j: usize) {
+        self.items.swap(i, j);
+        self.items[i].index = i;
+        self.items[j].index = j;
+    }
+
+    pub fn push(&mut self, entry: &Entry) -> Result<()> {
+        if self.has(&entry.id)? {
+            return Ok(());
+        }
+        self.items.push_front(entry.to_owned());
+
+        // insert key only if it does not already exist.
+        self.lookup.insert(entry.id, entry.to_owned());
+
+        Ok(())
+    }
+
+    pub fn pop(&mut self) -> Result<Option<Entry>> {
+        Ok(self.items.pop_front())
+    }
+
+    pub fn pop_back(&mut self) -> Result<Option<Entry>> {
+        Ok(self.items.pop_back())
+    }
+
+    pub fn get(&self, id: &ids::Id) -> Result<Option<Entry>> {
+        match self.lookup.get(id){
+            Some(v) => Ok(Some(v.to_owned())),
+            None => Ok(None),
+        }
+    }
+
+    pub fn has(&self, id: &ids::Id) -> Result<bool> {
+        match self.get(id) {
+            Ok(resp) => match resp {
+                Some(_) => Ok(true),
+                None => Ok(false),
+            },
+            Err(e) => Err(e),
+        }
+    }
+}

mini-kvvm/src/mempool/mod.rs

@@ -0,0 +1,207 @@
+pub mod data;
+
+use std::{
+    io::Result,
+    sync::{Arc, RwLock},
+};
+
+use avalanche_types::ids;
+use tokio::sync::broadcast;
+
+use crate::chain::tx::tx::Transaction;
+
+use self::data::{Data, Entry};
+
+pub struct Mempool {
+    data: Arc<RwLock<Data>>,
+    pending_tx: broadcast::Sender<()>,
+    new_txs: Vec<Transaction>,
+}
+
+impl Mempool {
+    pub fn new(max_size: usize) -> Self {
+        // initialize broadcast channel
+        let (pending_tx, _rx): (broadcast::Sender<()>, broadcast::Receiver<()>) =
+            tokio::sync::broadcast::channel(1);
+        Self {
+            data: Arc::new(RwLock::new(Data::new(max_size))),
+            /// Channel of length one, which the mempool ensures has an item on
+            /// it as long as there is an unissued transaction remaining in [txs].
+            pending_tx,
+            new_txs: Vec::new(),
+        }
+    }
+
+    /// Returns a broadcast receiver for the pending tx channel.
+    pub fn subscribe_pending(&self) -> broadcast::Receiver<()> {
+        self.pending_tx.subscribe()
+    }
+
+    /// Returns Tx from Id if it exists.
+    pub fn get(&self, id: &ids::Id) -> Result<Option<Transaction>> {
+        let data = self.data.read().unwrap();
+        if let Some(entry) = data.get(id)? {
+            if let Some(tx) = entry.tx {
+                return Ok(Some(tx));
+            }
+        }
+        Ok(None)
+    }
+
+    /// Adds a Tx Entry to mempool and writes to the pending channel.
+    pub fn add(&mut self, tx: Transaction) -> Result<bool> {
+        let tx_id = &tx.id;
+
+        let mut data = self.data.write().unwrap();
+        if data.has(tx_id)? {
+            return Ok(false);
+        }
+        let old_len = data.len();
+
+        let entry = &Entry {
+            id: tx_id.to_owned(),
+            tx: Some(tx.clone()),
+            index: old_len,
+        };
+
+        // Optimistically add tx to mempool
+        data.push(entry)?;
+
+        self.new_txs.push(tx);
+
+        self.add_pending();
+
+        Ok(true)
+    }
+
+    /// Returns len of mempool data.
+    pub fn len(&self) -> usize {
+        let data = self.data.read().unwrap();
+        data.len()
+    }
+
+    pub fn is_empty(&self) -> bool {
+         let data = self.data.read().unwrap();
+        data.is_empty()
+    }
+
+    /// Prunes any Ids not included in valid hashes set.
+    pub fn prune(&self, valid_hashes: ids::Set) {
+        let mut to_remove: Vec<ids::Id> = Vec::with_capacity(valid_hashes.len());
+
+        let data = self.data.write().unwrap();
+
+        for entry in data.items.iter() {
+            if let Some(tx) = &entry.tx {
+                if !valid_hashes.contains(&tx.id) {
+                    to_remove.push(entry.id);
+                }
+            }
+        }
+        // drop write lock
+        drop(data);
+
+        for id in to_remove.iter() {
+            log::debug!("attempting to prune id: {}", id);
+            if self.remove(id.to_owned()).is_some() {
+                log::debug!("id deleted: {}", id);
+            } else {
+                log::debug!("failed to delete id: {}: not found", id);
+            }
+        }
+    }
+
+    /// Removes Tx entry from mempool data if it exists.
+    pub fn remove(&self, id: ids::Id) -> Option<Transaction> {
+        let mut data = self.data.write().unwrap();
+
+        // TODO: try to optimize.
+        // find the position of the entry in vec and remove
+        match data.items.iter().position(|e| e.id == id) {
+            Some(index) => {
+                data.items.remove(index);
+            }
+            None => return None,
+        }
+
+        // remove entry from lookup
+        match data.lookup.remove(&id) {
+            Some(entry) => entry.tx,
+            None => {
+                // should not happen
+                log::error!("failed to remove id: {}: mempool is out of balance", id);
+                None
+            }
+        }
+    }
+
+    fn add_pending(&self) {
+        self.pending_tx.send(()).unwrap();
+    }
+}
+
+#[tokio::test]
+async fn test_mempool() {
+    use crate::chain::tx::{tx::TransactionType, unsigned};
+
+    // init mempool
+    let mut mempool = Mempool::new(10);
+    let mut pending_rx = mempool.subscribe_pending();
+
+    // create tx_1
+    let tx_data_1 = unsigned::TransactionData {
+        typ: TransactionType::Bucket,
+        bucket: "foo".to_string(),
+        key: "".to_string(),
+        value: vec![],
+    };
+    let resp = tx_data_1.decode();
+    assert!(resp.is_ok());
+    let utx_1 = resp.unwrap();
+    let tx_1 = Transaction::new(utx_1);
+
+    // add tx_1 to mempool
+    let tx_1_id = tx_1.id;
+    assert_eq!(mempool.add(tx_1).unwrap(), true);
+    // drain channel
+    let resp = pending_rx.recv().await;
+    assert!(resp.is_ok());
+    assert_eq!(mempool.len(), 1);
+
+    // add tx_1 as valid
+    let mut valid_txs = ids::new_set(2);
+    valid_txs.insert(tx_1_id);
+
+    // create tx_2
+    let tx_data_2 = unsigned::TransactionData {
+        typ: TransactionType::Bucket,
+        bucket: "bar".to_string(),
+        key: "".to_string(),
+        value: vec![],
+    };
+    let resp = tx_data_2.decode();
+    assert!(resp.is_ok());
+    let utx_2 = resp.unwrap();
+    let mut tx_2 = Transaction::new(utx_2);
+    tx_2.id = ids::Id::from_slice("sup".as_bytes());
+
+    // add tx_2 to mempool
+    assert_eq!(mempool.add(tx_2).unwrap(), true);
+    assert_eq!(mempool.len(), 2);
+
+    // drain channel
+    let resp = pending_rx.recv().await;
+    assert!(resp.is_ok());
+
+    // prune tx_2 as invalid
+    mempool.prune(valid_txs);
+
+    // verify one tx entry removed
+    assert_eq!(mempool.len(), 1);
+
+    // verify tx_1 exists
+    let resp = mempool.get(&tx_1_id);
+    assert!(resp.is_ok());
+
+    assert_eq!(resp.unwrap().unwrap().id, tx_1_id);
+}