Skip to content
This repository was archived by the owner on Jan 22, 2025. It is now read-only.

Add execution batch stats #18096

Merged
merged 1 commit into from
Jun 22, 2021
Merged

Add execution batch stats #18096

merged 1 commit into from
Jun 22, 2021

Conversation

sakridge
Copy link
Contributor

Problem

Batch execution size not very visible from metrics.

Summary of Changes

Add the number of executed batches and the total batch size

Fixes #

@sakridge sakridge requested a review from jeffwashington June 21, 2021 15:41
Copy link
Contributor

@jeffwashington jeffwashington left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@sakridge sakridge added the CI Pull Request is ready to enter CI label Jun 21, 2021
@solana-grimes solana-grimes removed the CI Pull Request is ready to enter CI label Jun 21, 2021
@codecov
Copy link

codecov bot commented Jun 21, 2021

Codecov Report

Merging #18096 (96d6b29) into master (d0e1349) will increase coverage by 0.0%.
The diff coverage is 75.0%.

@@           Coverage Diff           @@
##           master   #18096   +/-   ##
=======================================
  Coverage    82.3%    82.3%           
=======================================
  Files         434      434           
  Lines      121192   121205   +13     
=======================================
+ Hits        99771    99808   +37     
+ Misses      21421    21397   -24     

@sakridge sakridge merged commit e808f34 into solana-labs:master Jun 22, 2021
@sakridge sakridge added the v1.7 label Jun 22, 2021
@sakridge sakridge deleted the batch-stats branch June 22, 2021 13:23
mergify bot pushed a commit that referenced this pull request Jun 22, 2021
(cherry picked from commit e808f34)
jeffwashington pushed a commit that referenced this pull request Jun 23, 2021
(cherry picked from commit e808f34)

Co-authored-by: sakridge <sakridge@gmail.com>
@brooksprumo brooksprumo mentioned this pull request Aug 23, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants