Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add status logging while processing a ledger #3916

Merged
merged 1 commit into from
Apr 21, 2019
Merged

Conversation

mvines
Copy link
Member

@mvines mvines commented Apr 21, 2019

Problem

process_blocktree() can take minutes, or worse, when processing a large ledger (cc: #2475) with no output of any kind to the user.

Summary of Changes

Every two seconds process_blocktree() now emits an info! log containing the current block it's processing.

@codecov
Copy link

codecov bot commented Apr 21, 2019

Codecov Report

Merging #3916 into master will increase coverage by <.1%.
The diff coverage is 50%.

@@           Coverage Diff            @@
##           master   #3916     +/-   ##
========================================
+ Coverage    77.8%   77.8%   +<.1%     
========================================
  Files         162     162             
  Lines       27238   27242      +4     
========================================
+ Hits        21202   21207      +5     
+ Misses       6036    6035      -1

1 similar comment
@codecov
Copy link

codecov bot commented Apr 21, 2019

Codecov Report

Merging #3916 into master will increase coverage by <.1%.
The diff coverage is 50%.

@@           Coverage Diff            @@
##           master   #3916     +/-   ##
========================================
+ Coverage    77.8%   77.8%   +<.1%     
========================================
  Files         162     162             
  Lines       27238   27242      +4     
========================================
+ Hits        21202   21207      +5     
+ Misses       6036    6035      -1

@mvines mvines merged commit af9ebf1 into solana-labs:master Apr 21, 2019
@mvines mvines deleted the l branch April 21, 2019 03:18
danpaul000 pushed a commit to danpaul000/solana that referenced this pull request Apr 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant