Skip to content

Commit

Permalink
Remove unused checkpoint metrics (MystenLabs#12863)
Browse files Browse the repository at this point in the history
## Description 

as title

## Test Plan 

eyes

---
If your changes are not user-facing and not a breaking change, you can
skip the following section. Otherwise, please indicate what changed, and
then add to the Release Notes section as highlighted during the release
process.

### Type of Change (Check all that apply)

- [ ] protocol change
- [ ] user-visible impact
- [ ] breaking change for a client SDKs
- [ ] breaking change for FNs (FN binary must upgrade)
- [ ] breaking change for validators or node operators (must upgrade
binaries)
- [ ] breaking change for on-chain data layout
- [ ] necessitate either a data wipe or data migration

### Release notes
Remove unused checkpoint metrics
  • Loading branch information
longbowlu authored Jul 7, 2023
1 parent c641137 commit 636e770
Showing 1 changed file with 0 additions and 14 deletions.
14 changes: 0 additions & 14 deletions crates/sui-core/src/checkpoints/metrics.rs
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,6 @@ pub struct CheckpointMetrics {
pub last_certified_checkpoint: IntGauge,
pub last_constructed_checkpoint: IntGauge,
pub checkpoint_errors: IntCounter,
pub builder_utilization: IntCounter,
pub aggregator_utilization: IntCounter,
pub transactions_included_in_checkpoint: IntCounter,
pub checkpoint_roots_count: IntCounter,
pub checkpoint_participation: IntCounterVec,
Expand Down Expand Up @@ -58,18 +56,6 @@ impl CheckpointMetrics {
registry
)
.unwrap(),
builder_utilization: register_int_counter_with_registry!(
"builder_utilization",
"Checkpoints builder task utilization",
registry
)
.unwrap(),
aggregator_utilization: register_int_counter_with_registry!(
"aggregator_utilization",
"Checkpoints aggregator task utilization",
registry
)
.unwrap(),
transactions_included_in_checkpoint: register_int_counter_with_registry!(
"transactions_included_in_checkpoint",
"Transactions included in a checkpoint",
Expand Down

0 comments on commit 636e770

Please sign in to comment.