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

Rework fullnode boot sequence #2498

Merged
merged 1 commit into from
Jan 20, 2019
Merged

Rework fullnode boot sequence #2498

merged 1 commit into from
Jan 20, 2019

Conversation

mvines
Copy link
Contributor

@mvines mvines commented Jan 20, 2019

The fullnode/src/main.rs boot sequence has been generalized such that the bootstrap leader and other fullnodes now boot identically.

Also took the opportunity to improve error reporting, rename local variables to be more descriptive, clarified log messages, and refactored the operations in main() to be grouped more intuitively.

Fixes #2493

@codecov
Copy link

codecov bot commented Jan 20, 2019

Codecov Report

Merging #2498 into master will decrease coverage by 1.2%.
The diff coverage is 4.5%.

@@           Coverage Diff            @@
##           master   #2498     +/-   ##
========================================
- Coverage    77.4%   76.2%   -1.3%     
========================================
  Files         110     110             
  Lines       17719   18008    +289     
========================================
- Hits        13728   13726      -2     
- Misses       3991    4282    +291

@mvines mvines merged commit 5b73a8e into solana-labs:master Jan 20, 2019
@mvines mvines deleted the boot branch January 20, 2019 05:35
);
}
None => {
info!("leader ready... rpc address: {}", rpc_addr);
Copy link
Contributor

Choose a reason for hiding this comment

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

It will print "leader ready..." when it's ready to receive transactions.

https://github.com/solana-labs/solana/blob/978b26a9c/docs/src/cluster/bench-tps.md

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