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

Resume Sync After Initial Sync Completes #426

Closed
nisdas opened this issue Aug 22, 2018 · 2 comments
Closed

Resume Sync After Initial Sync Completes #426

nisdas opened this issue Aug 22, 2018 · 2 comments
Labels
Priority: Medium Medium priority item
Milestone

Comments

@nisdas
Copy link
Member

nisdas commented Aug 22, 2018

With #404 being merged in , initial sync and resume sync are separate services. With this when initial sync is done, it exits the initial sync service. However normal sync is not resumed. A PR dealing with this issue will allow normal sync to resume, also the PR will deal with changing the sync polling interval so that nodes with insufficient number of peers are given a larger interval to get up to sync

  • Resume Sync after Initial Sync
  • Make sync polling interval dependent on number of connected peers
@nisdas nisdas added this to the Ruby milestone Aug 22, 2018
@nisdas nisdas added this to To do in Beacon Chain via automation Aug 22, 2018
@rauljordan rauljordan changed the title Resume Sync after Initial Sync completes Resume Sync after Initial Sync Completes Aug 22, 2018
@rauljordan rauljordan changed the title Resume Sync after Initial Sync Completes Resume Sync After Initial Sync Completes Aug 22, 2018
@nisdas nisdas self-assigned this Aug 26, 2018
@rawfalafel rawfalafel assigned rawfalafel and unassigned nisdas Sep 5, 2018
@nisdas nisdas mentioned this issue Sep 22, 2018
prestonvanloon added a commit to prestonvanloon/prysm that referenced this issue Sep 22, 2018
prestonvanloon added a commit that referenced this issue Sep 22, 2018
* Add TODO link for issue #555

* Remove database related TODO

* update todo for issue #426
@rawfalafel rawfalafel removed their assignment Oct 9, 2018
@rauljordan rauljordan added Priority: Medium Medium priority item and removed Beacon Chain labels Oct 9, 2018
@rauljordan
Copy link
Contributor

Can this be closed @nisdas?

@nisdas
Copy link
Member Author

nisdas commented Nov 21, 2018

@rauljordan , Yeap this is resolved by #787

@nisdas nisdas closed this as completed Nov 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: Medium Medium priority item
Projects
No open projects
Beacon Chain
  
To do
Development

No branches or pull requests

4 participants