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

[WIP] Revert self-sampling #433

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

hasyimibhar
Copy link
Contributor

@hasyimibhar hasyimibhar commented Dec 18, 2019

This PR experiments with the disabling of self-sampling of block by each node, which was introduced in PR #428. This is done as part of an effort to debug why the consensus is stalling under local benchmark with K=2 and N > 3.

You can find the debug dump for a sample run here:
node1.txt
node2.txt
node3.txt
node4.txt

@hasyimibhar hasyimibhar self-assigned this Dec 18, 2019
@AhmadMuzakkir
Copy link
Contributor

AhmadMuzakkir commented Dec 18, 2019

This PR still stalls on my local.

My testing stalled at block 145.
I tested 4 nodes, k = 2, with database.

Below are the commands I used to run the nodes.

go run -tags=debug . -port 3000 -api.port 9000 -wallet config/wallet.txt -db /private/tmp/wavelet/node1 -loglevel debug -sys.snowball.k 2 |& tee ~/Desktop/443/node1.tx

go run . -port 3001 -api.port 9001 -wallet config/wallet2.txt -db /private/tmp/wavelet/node2 -loglevel debug -sys.snowball.k 2 127.0.0.1:3000 |& tee ~/Desktop/443/node2.txt 127.0.0.1:3000

go run -tags=debug . -port 3002 -api.port 9002 -wallet config/wallet3.txt -db /private/tmp/wavelet/node3 -loglevel debug -sys.snowball.k 2 127.0.0.1:3000 |& tee ~/Desktop/443/node3.txt 127.0.0.1:3000

go run -tags=debug . -port 3003 -api.port 9003 -db /private/tmp/wavelet/node4 -loglevel debug -sys.snowball.k 2 127.0.0.1:3000 |& tee ~/Desktop/443/node4.txt

Note: I forgot to put tags=debug on Node 2.

For some reason I could not upload CLI dumps to this PR.

I put them in https://drive.google.com/drive/folders/1GHHM-bN4LwPm3QrNQBn-yslTAEVD6ns5?usp=sharing

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.

None yet

2 participants