Do not use binary_search, do simple compare in one loop

core/src/banking_stage/consumer.rs

@@ -208,10 +208,13 @@ impl Consumer {
 
         // Now we track the performance for the interested transactions which is not in the retryable_transaction_indexes
         // We assume the retryable_transaction_indexes is already sorted.
+        let mut retryable_idx = 0;
         for (index, packet) in packets_to_process.iter().enumerate() {
             if packet.original_packet().meta().is_perf_track_packet() {
                 if let Some(start_time) = packet.start_time() {
-                    if retryable_transaction_indexes.binary_search(&index).is_err() {
+                    if retryable_idx >= retryable_transaction_indexes.len()
+                        || retryable_transaction_indexes[retryable_idx] != index
+                    {
                         let duration = Instant::now().duration_since(*start_time);
 
                         debug!(
@@ -222,6 +225,10 @@ impl Consumer {
                             "txn-metrics-banking-stage-process-us",
                             duration.as_micros() as usize
                         );
+                    } else {
+                        // This packet is retried, advance the retry index to the next, as the next packet's index will
+                        // certainly be > than this.
+                        retryable_idx += 1;
                     }
                 }
             }