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

generate_index timings #17983

Merged
merged 1 commit into from
Jun 17, 2021
Merged

Conversation

jeffwashington
Copy link
Contributor

@jeffwashington jeffwashington commented Jun 15, 2021

Problem

We are about to rework and pipeline generate_index with other stages of startup.

Summary of Changes

separate timings for generate_index into its own struct, like other places we capture timings.
Fixes #

@codecov
Copy link

codecov bot commented Jun 15, 2021

Codecov Report

Merging #17983 (98fafad) into master (f558b9b) will increase coverage by 0.0%.
The diff coverage is 72.7%.

@@           Coverage Diff           @@
##           master   #17983   +/-   ##
=======================================
  Coverage    82.6%    82.6%           
=======================================
  Files         435      435           
  Lines      121576   121580    +4     
=======================================
+ Hits       100468   100481   +13     
+ Misses      21108    21099    -9     

@jeffwashington jeffwashington requested a review from jbiseda June 16, 2021 13:35
@jeffwashington jeffwashington marked this pull request as ready for review June 16, 2021 13:36
Copy link
Contributor

@jbiseda jbiseda left a comment

Choose a reason for hiding this comment

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

Would it be useful to add a trait for report() calls used by similar structures? Perhaps as a separate issue.

@jeffwashington jeffwashington merged commit 1feaaf0 into solana-labs:master Jun 17, 2021
mergify bot pushed a commit that referenced this pull request Jun 17, 2021
(cherry picked from commit 1feaaf0)
jeffwashington added a commit that referenced this pull request Jun 17, 2021
(cherry picked from commit 1feaaf0)
jeffwashington added a commit that referenced this pull request Jun 17, 2021
(cherry picked from commit 1feaaf0)
jeffwashington added a commit that referenced this pull request Jun 21, 2021
(cherry picked from commit 1feaaf0)

Co-authored-by: Jeff Washington (jwash) <[email protected]>
@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.

2 participants