Skip to content

Commit 2c57d3a

Browse files
authored
turbine: Combine counter into existing datapoint (#10128)
1 parent 947617c commit 2c57d3a

File tree

4 files changed

+6
-3
lines changed

4 files changed

+6
-3
lines changed

ledger/src/shred/stats.rs

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,7 @@ pub struct ProcessShredsStats {
3838
pub err_unknown_chained_merkle_root: u64,
3939
pub(crate) padding_bytes: usize,
4040
pub(crate) data_bytes: usize,
41+
pub(crate) num_entries: usize,
4142
num_merkle_data_shreds: usize,
4243
num_merkle_coding_shreds: usize,
4344
}
@@ -87,6 +88,7 @@ impl ProcessShredsStats {
8788
("slot", slot, i64),
8889
("shredding_time", self.shredding_elapsed, i64),
8990
("receive_time", self.receive_elapsed, i64),
91+
("num_entries", self.num_entries, i64),
9092
("num_data_shreds", self.num_merkle_data_shreds, i64),
9193
("num_coding_shreds", self.num_merkle_coding_shreds, i64),
9294
("slot_broadcast_time", slot_broadcast_time, i64),
@@ -241,6 +243,7 @@ impl AddAssign<ProcessShredsStats> for ProcessShredsStats {
241243
err_unknown_chained_merkle_root,
242244
padding_bytes,
243245
data_bytes,
246+
num_entries,
244247
num_merkle_data_shreds,
245248
num_merkle_coding_shreds,
246249
} = rhs;
@@ -260,6 +263,7 @@ impl AddAssign<ProcessShredsStats> for ProcessShredsStats {
260263
self.err_unknown_chained_merkle_root += err_unknown_chained_merkle_root;
261264
self.padding_bytes += padding_bytes;
262265
self.data_bytes += data_bytes;
266+
self.num_entries += num_entries;
263267
self.num_merkle_data_shreds += num_merkle_data_shreds;
264268
self.num_merkle_coding_shreds += num_merkle_coding_shreds;
265269
for (i, bucket) in self.num_data_shreds_hist.iter_mut().enumerate() {

ledger/src/shredder.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -75,6 +75,7 @@ impl Shredder {
7575
reed_solomon_cache: &ReedSolomonCache,
7676
stats: &mut ProcessShredsStats,
7777
) -> impl Iterator<Item = Shred> + use<> {
78+
stats.num_entries += entries.len();
7879
let now = Instant::now();
7980
let entries = wincode::serialize(entries).unwrap();
8081
stats.serialize_elapsed += now.elapsed().as_micros() as u64;

turbine/src/broadcast_stage.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ use {
2222
solana_keypair::Keypair,
2323
solana_ledger::{blockstore::Blockstore, shred::Shred},
2424
solana_measure::measure::Measure,
25-
solana_metrics::{inc_new_counter_error, inc_new_counter_info},
25+
solana_metrics::inc_new_counter_error,
2626
solana_net_utils::SocketAddrSpace,
2727
solana_poh::poh_recorder::WorkingBankEntry,
2828
solana_pubkey::Pubkey,

turbine/src/broadcast_stage/standard_broadcast_run.rs

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -190,8 +190,6 @@ impl StandardBroadcastRun {
190190
last_tick_height,
191191
} = receive_results;
192192

193-
inc_new_counter_info!("broadcast_service-entries_received", entries.len());
194-
195193
let mut to_shreds_time = Measure::start("broadcast_to_shreds");
196194

197195
if self.slot != bank.slot() {

0 commit comments

Comments
 (0)