farmer: Fix review comments (tracing errors).

subspace · Jul 28, 2022 · fb1687f · fb1687f
1 parent 8ec51dc
commit fb1687f
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/crates/subspace-farmer/src/dsn.rs b/crates/subspace-farmer/src/dsn.rs
@@ -266,7 +266,7 @@ where
                 .unzip();
 
             if let Err(err) = on_sync.on_pieces(pieces, piece_indexes).await {
-                error!("DSN sync process returned an error: {}", err);
+                error!(%err, "DSN sync process returned an error");
             }
         }
     }

diff --git a/crates/subspace-farmer/src/single_plot_farm.rs b/crates/subspace-farmer/src/single_plot_farm.rs
@@ -716,7 +716,6 @@ struct VerifyingPlotter<RC> {
 }
 
 impl<RC: RpcClient> VerifyingPlotter<RC> {
-    //TODO: Optimize: parallel execution, batch execution.
     /// Verifies pieces against the blockchain.
     pub async fn verify_pieces_at_blockchain(
         &self,
@@ -751,7 +750,7 @@ impl<RC: RpcClient> VerifyingPlotter<RC> {
                 if let Some(root) = root {
                     Ok(root)
                 } else {
-                    error!(?piece_index, "No records root found for piece_index.");
+                    error!(piece_index, "No records root found for piece_index.");
 
                     Err(PiecesVerificationError::NoRecordsRootFound)
                 }