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.

Sign up for GitHub

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

Jump to bottom

fix: puller rewrite and bug fixes #3437

Merged
merged 24 commits into from Oct 25, 2022
Merged

fix: puller rewrite and bug fixes #3437

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
c66214a
fix: bugs
istae Oct 14, 2022
84b7a37
fix: bugs
istae Oct 14, 2022
20cbcdf
fix: bugs
istae Oct 15, 2022
6338679
chore: more bugs
istae Oct 17, 2022
e15ce06
fix: more bugs
istae Oct 18, 2022
bcc76d4
fix: puller rewrite
istae Oct 18, 2022
bd72ec0
fix: trigger fix
istae Oct 18, 2022
488db7b
chore: comments
istae Oct 18, 2022
8a75cc4
chore: lint
istae Oct 18, 2022
987a878
chore: metrics
istae Oct 19, 2022
a03b64d
fix: prune gone peers
istae Oct 19, 2022
32ac92a
fix: missing unlock
istae Oct 19, 2022
a95ab4f
fix: removed listen on quite
istae Oct 19, 2022
740c436
fix: disconnect with swarm address
istae Oct 19, 2022
081e428
fix: pullsync and puller sync process to continue (#3444)
istae Oct 20, 2022
ff7688f
revert: falling edge
istae Oct 20, 2022
a6a7ca9
fix: missing metric
istae Oct 20, 2022
0f53bfc
fix: context
istae Oct 20, 2022
c3fb373
chore: tests
istae Oct 20, 2022
239a691
fix: remove unncessary disconnect
istae Oct 20, 2022
1061c6c
fix: timer
istae Oct 20, 2022
6abf7a4
chore: retry sleep
istae Oct 23, 2022
a7fe7a9
chore: peer gone test
istae Oct 24, 2022
ae7a090
chore: puller sync sleep dur options
istae Oct 25, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
1 change: 0 additions & 1 deletion .golangci.yml
View file
Open in desktop
Expand Up @@ -23,7 +23,6 @@ linters:
- importas
- ineffassign
- misspell
- nakedret
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have no preference on naked vs not-naked returns, but could not see a strong enough need in this pr to change the linter configuration, and consequently the coding policy. The changes with naked returns have no functional requirement, while naked returns could be a source of a subtle problems.

- nilerr
- noctx
- paralleltest
Expand Down
2 changes: 1 addition & 1 deletion pkg/node/node.go
View file
Open in desktop
Expand Up @@ -931,7 +931,7 @@ func NewBee(interrupt chan struct{}, sysInterrupt chan os.Signal, addr string, p

var pullerService *puller.Puller
if o.FullNodeMode && !o.BootnodeMode {
pullerService = puller.New(stateStore, kad, batchStore, pullSyncProtocol, logger, puller.Options{}, warmupTime)
pullerService = puller.New(stateStore, kad, batchStore, pullSyncProtocol, logger, puller.Options{SyncSleepDur: puller.DefaultSyncSleepDur}, warmupTime)
istae marked this conversation as resolved.
Show resolved Hide resolved
b.pullerCloser = pullerService
}

Expand Down