Skip to content
This repository has been archived by the owner on Aug 2, 2021. It is now read-only.

[WIP] Push syncing pss #971

Closed
wants to merge 3 commits into from
Closed

[WIP] Push syncing pss #971

wants to merge 3 commits into from

Conversation

zelig
Copy link
Member

@zelig zelig commented Oct 24, 2018

push sync implemented as described in https://hackmd.io/ffBjSu8RTyKikvRO7bYrzA#

temporary PR for debugging
try running this and see that often the neighbourhood sends are not delivered for some reason.

go test -v -cpu 8  ./swarm/syncer/ -count 100 -run TestSyncerWithPss -loglevel 1

if you change this line https://github.com/ethersphere/go-ethereum/pull/971/files#diff-f499799c287a40bb411f478314862498R219
to chunkCnt := 1
it always works

@zelig zelig self-assigned this Oct 24, 2018
@zelig zelig requested a review from nolash October 24, 2018 18:15
@zelig zelig requested a review from nonsense as a code owner October 24, 2018 18:15
This was referenced Nov 8, 2018
@nolash nolash force-pushed the pss-handler-refactor-depth branch from 10def86 to 11f1528 Compare November 9, 2018 10:30
@adamschmideg adamschmideg added this to the 2018 Christmas Edition milestone Nov 15, 2018
@nolash nolash force-pushed the pss-handler-refactor-depth branch from a46ac96 to b13b05c Compare November 20, 2018 12:18
@zelig zelig mentioned this pull request Nov 26, 2018
@zelig zelig changed the base branch from pss-handler-refactor-depth to master November 28, 2018 13:54
@zelig zelig removed the request for review from acud November 28, 2018 13:55
@zelig zelig mentioned this pull request Nov 29, 2018
20 tasks
@zelig zelig force-pushed the push-syncing-pss branch from f5327ab to 30157fd Compare January 5, 2019 10:29
@zelig
Copy link
Member Author

zelig commented Jan 5, 2019

this now works as expected with current master.

tags and db interface needs to be replaced according to #1028

@frncmx frncmx added the on hold label Jan 7, 2019
@frncmx
Copy link
Contributor

frncmx commented Jan 7, 2019

blocked by #987

@zelig zelig mentioned this pull request Apr 1, 2019
3 tasks
@zelig
Copy link
Member Author

zelig commented Apr 1, 2019

obsoleted in favour of #1323

@zelig zelig closed this Apr 1, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants