From 75eda5341fd4543546068ad642383bbf9e67a02f Mon Sep 17 00:00:00 2001
From: Agnish Ghosh <iamagnix@gmail.com>
Date: Wed, 2 Oct 2024 13:41:54 +0530
Subject: [PATCH] adding more logging around db

---
 beacon_chain/gossip_processing/block_processor.nim   | 2 ++
 beacon_chain/gossip_processing/gossip_validation.nim | 1 -
 beacon_chain/nimbus_beacon_node.nim                  | 6 ++++++
 3 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/beacon_chain/gossip_processing/block_processor.nim b/beacon_chain/gossip_processing/block_processor.nim
index f9ff552bf..bfc640163 100644
--- a/beacon_chain/gossip_processing/block_processor.nim
+++ b/beacon_chain/gossip_processing/block_processor.nim
@@ -258,6 +258,8 @@ proc storeBackfillBlock(
   let data_columns = dataColumnsOpt.valueOr: DataColumnSidecars @[]
   for c in data_columns:
     self.consensusManager.dag.db.putDataColumnSidecar(c[])
+    debug "Data column written to database!",
+      data_column = shortLog(c[])
 
   res
 
diff --git a/beacon_chain/gossip_processing/gossip_validation.nim b/beacon_chain/gossip_processing/gossip_validation.nim
index 96ccf873f..f304529b0 100644
--- a/beacon_chain/gossip_processing/gossip_validation.nim
+++ b/beacon_chain/gossip_processing/gossip_validation.nim
@@ -528,7 +528,6 @@ proc validateDataColumnSidecar*(
       block_header.slot, block_header.proposer_index, data_column_sidecar.index):
     return errIgnore("DataColumnSidecar: already have valid data column from same proposer")
 
-
   # [REJECT] The sidecar's column data is valid as 
   # verified by `verify_data_column_kzg_proofs(sidecar)`
   block:
diff --git a/beacon_chain/nimbus_beacon_node.nim b/beacon_chain/nimbus_beacon_node.nim
index 76b53a254..fab5e85e7 100644
--- a/beacon_chain/nimbus_beacon_node.nim
+++ b/beacon_chain/nimbus_beacon_node.nim
@@ -1531,6 +1531,9 @@ proc tryReconstructingDataColumns* (self: BeaconNode,
     data_column_sidecars.add data_column[]
     storedColumns.add data_column.index
 
+    debugEcho "Pre stored columns"
+    debugEcho storedColumns
+
   # storedColumn number is less than the NUMBER_OF_COLUMNS
   # then reconstruction is not possible, and if all the data columns
   # are already stored then we do not need to reconstruct at all
@@ -1549,6 +1552,8 @@ proc tryReconstructingDataColumns* (self: BeaconNode,
 
       finalisedDataColumns.add(data_column)
       db.putDataColumnSidecar(data_column)
+      debug "Reconstructed data column written to database",
+        data_column = shortLog(data_column)
   ok(finalisedDataColumns)
 
 proc reconstructAndSendDataColumns*(node: BeaconNode) {.async.} =
@@ -1569,6 +1574,7 @@ proc reconstructAndSendDataColumns*(node: BeaconNode) {.async.} =
         var
           das_workers = newSeq[Future[SendResult]](dc.len)
         for i in 0..<dc.lenu64:
+          debugEcho "Computing subnet before broadcasting reconstructed data columns"
           let subnet_id = compute_subnet_for_data_column_sidecar(dc[i].index)
           das_workers[i] =
               node.network.broadcastDataColumnSidecar(subnet_id, dc[i])