NDEV-2183: Make EthSyncing fields public

neonlabsorg · Sep 14, 2023 · 75f7760 · 75f7760
1 parent e7be346
commit 75f7760
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 5 deletions.
diff --git a/evm_loader/lib/src/types/tracer_ch_common.rs b/evm_loader/lib/src/types/tracer_ch_common.rs
@@ -125,7 +125,7 @@ impl EthSyncStatus {
 #[derive(Row, Debug, Deserialize, Serialize)]
 #[serde(rename_all = "camelCase")]
 pub struct EthSyncing {
-    starting_block: u64,
-    current_block: u64,
-    highest_block: u64,
+    pub starting_block: u64,
+    pub current_block: u64,
+    pub highest_block: u64,
 }
diff --git a/evm_loader/lib/src/types/tracer_ch_db.rs b/evm_loader/lib/src/types/tracer_ch_db.rs
@@ -506,7 +506,7 @@ impl ClickHouseDb {
         }
     }
 
-    pub async fn get_slot_by_blockhash(&self, blockhash: &str) -> ChResult<Option<u64>> {
+    pub async fn get_slot_by_blockhash(&self, blockhash: &str) -> ChResult<u64> {
         let query = r#"SELECT slot
         FROM events.notify_block_distributed
         WHERE hash = ?
@@ -521,7 +521,15 @@ impl ClickHouseDb {
                 .await,
         )?;
 
-        Ok(slot)
+        match slot {
+            Some(slot) => return Ok(slot),
+            None => {
+                let err = clickhouse::error::Error::Custom(format!(
+                    "get_sync_status: no data available",
+                ));
+                return Err(ChError::Db(err));
+            }
+        }
     }
 
     pub async fn get_sync_status(&self) -> ChResult<EthSyncStatus> {