Revert "Add timing info to landmark bundle computation"

This reverts commit 643b8f7.

Author: Michael Rosenberg

crates/mtc_worker/src/sequencer_do.rs

@@ -3,7 +3,7 @@
 
 //! Sequencer is the 'brain' of the CT log, responsible for sequencing entries and maintaining log state.
 
-use std::time::Duration;
+use std::{future::Future, pin::Pin, time::Duration};
 
 use crate::{
     frontend_worker::{LandmarkBundle, SubtreeWithConsistencyProof},
@@ -155,10 +155,7 @@ async fn get_landmark_subtrees(
 ) -> Result<Vec<SubtreeWithConsistencyProof>> {
     let cached_object_backend = CachedRoObjectBucket::new(ObjectBucket::new(bucket));
     let mut subtrees = Vec::new();
-    let start = Date::now().as_millis();
-    let it = landmark_sequence.subtrees().collect::<Vec<_>>();
-    let num_subtrees = it.len();
-    for landmark_subtree in it {
+    for landmark_subtree in landmark_sequence.subtrees() {
         let (consistency_proof, landmark_subtree_hash) = match prove_subtree_consistency(
             checkpoint_hash,
             checkpoint_size,
@@ -178,12 +175,6 @@ async fn get_landmark_subtrees(
             consistency_proof: consistency_proof.iter().map(|h| h.0).collect(),
         });
     }
-    let end = Date::now().as_millis();
-    log::error!("num subtrees is {num_subtrees}");
-    log::error!(
-        "get subtrees was {:.02}ms per subtree",
-        (end - start) as f64 / num_subtrees as f64
-    );
 
     Ok(subtrees)
 }