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

Stronger faster better #17

Merged
merged 1 commit into from
Jun 6, 2016
Merged

Stronger faster better #17

merged 1 commit into from
Jun 6, 2016

Conversation

dignifiedquire
Copy link
Member

Found issues when distributing blocks in networks with >= 3 nodes. This adds tests for that and hopefully a fix soon :)

cc @diasdavid @haadcode

@dignifiedquire
Copy link
Member Author

Mostly works, but socket hung up errors are making life hard :/

@daviddias
Copy link
Member

Pasting the notes from the etherpad. Later we should do also the same tests on full js-ipfs nodes, so that we account for every possibility

bitswap tests

  • script to spawn a node, given a config file, creates a random file every X seconds and prints the hash (so that it can be requested
  • spawn 1 node on the main thread with TCP
  • dial to the other down
  • fetch all of the blocks the other node outputs
  • spawn 5 nodes in separate process and repeat the above
  • spawn 10 nodes and ..
  • spawn 20 nodes
  • repeat the previous proc
  • note: measure time (avg, max and min) of fetching the block and memory usage in the beginning and end

timing data in different events

  • use logs (debug)

@dignifiedquire
Copy link
Member Author

@diasdavid as not all of this will happen in this PR I moved the notes to #8

@daviddias
Copy link
Member

@dignifiedquire dignifiedquire changed the title [WIP] Stronger faster better Stronger faster better Jun 6, 2016
- send messages to all connected peers on start up
- ensure b58 endcoded strings are used everyhwere
- update dependencies
- add improved network tests
@dignifiedquire dignifiedquire merged commit fc128b0 into master Jun 6, 2016
@dignifiedquire dignifiedquire deleted the stronger-faster-better branch June 6, 2016 11:06
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