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

V0.12.1.x Fixes for sync: #573

Merged
merged 1 commit into from
Sep 2, 2015
Merged

V0.12.1.x Fixes for sync: #573

merged 1 commit into from
Sep 2, 2015

Conversation

UdjinM6
Copy link

@UdjinM6 UdjinM6 commented Aug 25, 2015

Fixes for sync:

  • lower MASTERNODE_SYNC_THRESHOLD (4->2)
  • higher wait MASTERNODE_SYNC_TIMEOUT*2 and fail MASTERNODE_SYNC_TIMEOUT*5 timeouts for each step
  • ask at most MASTERNODE_SYNC_THRESHOLD*3 nodes for sync data
  • sync again 1 minute after last fail

So we will ask at most 6 nodes and stop updating lastTime...s when we had data from at least 2 of them but will fail if we had nothing in 25 sec since current step start.
In case of fail resync again faster.

EDIT: updated

- lower MASTERNODE_SYNC_THRESHOLD (4->2)
- higher wait MASTERNODE_SYNC_TIMEOUT*2  and fail  MASTERNODE_SYNC_TIMEOUT*5 timeouts for each step
- ask at most MASTERNODE_SYNC_THRESHOLD*3 nodes for sync data
- sync again 1 minute after last fail

So we will ask at most 6 nodes and stop updating lastTime...s when we had data from at least 2 of them but will fail if we had nothing in 25 sec since current step start.
In case of fail resync again faster.
eduffield222 pushed a commit that referenced this pull request Sep 2, 2015
@eduffield222 eduffield222 merged commit b4d3de4 into dashpay:v0.12.1.x Sep 2, 2015
@UdjinM6 UdjinM6 deleted the v0.12.1.x_fix_sync branch March 7, 2016 03:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants