review fixes

src/index_controller/local_index_controller/index_store.rs

@@ -192,7 +192,7 @@ impl IndexStore {
         Ok((index, update_store))
     }
 
-    /// Same a get or create, but returns an error if the index already exists.
+    /// Same as `get_or_create`, but returns an error if the index already exists.
     pub fn create_index(
         &self,
         name: impl AsRef<str>,
@@ -219,23 +219,17 @@ impl IndexStore {
     /// Returns each index associated with it's metadata;
     pub fn list_indexes(&self) -> anyhow::Result<Vec<(String, IndexMeta)>> {
         let txn = self.env.read_txn()?;
-        let indexes = self.name_to_uuid
+        let metas = self.name_to_uuid
             .iter(&txn)?
             .filter_map(|entry| entry
-                .map_err(|e| {
+                .map_err(|e| { error!("error decoding entry while listing indexes: {}", e); e }).ok());
-                    error!("error decoding entry while listing indexes: {}", e);
+        let mut indexes = Vec::new();
-                    e
+        for (name, uuid) in metas {
-                })
+            let meta = self.uuid_to_index_meta
-                .ok())
+                .get(&txn, &uuid)?
-            .map(|(name, uuid)| {
+                .unwrap_or_else(|| panic!("corrupted database, index {} should exist.", name));
-                let meta = self.uuid_to_index_meta
+            indexes.push((name.to_owned(), meta));
-                    .get(&txn, &uuid)
+        }
-                    .ok()
-                    .flatten()
-                    .unwrap_or_else(|| panic!("corrupted database, index {} should exist.", name));
-                (name.to_owned(), meta)
-            })
-            .collect();
         Ok(indexes)
     }
 }

src/index_controller/mod.rs

@@ -25,7 +25,7 @@ pub struct IndexMetadata {
     uuid: Uuid,
     created_at: DateTime<Utc>,
     updated_at: DateTime<Utc>,
-    pub primary_key: Option<String>,
+    primary_key: Option<String>,
 }
 
 #[derive(Debug, Clone, Serialize, Deserialize)]