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

Make Status Requests Asynchronous #4577

Merged
merged 6 commits into from Jan 19, 2020
Merged

Make Status Requests Asynchronous #4577

merged 6 commits into from Jan 19, 2020

Conversation

nisdas
Copy link
Member

@nisdas nisdas commented Jan 18, 2020

Encloses the status request in a separate goroutine, so as to not block the parent thread

@codecov
Copy link

codecov bot commented Jan 18, 2020

Codecov Report

❗ No coverage uploaded for pull request base (master@3ae3703). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             master   #4577   +/-   ##
========================================
  Coverage          ?   27.4%           
========================================
  Files             ?     192           
  Lines             ?   13237           
  Branches          ?       0           
========================================
  Hits              ?    3627           
  Misses            ?    8977           
  Partials          ?     633

prestonvanloon
prestonvanloon previously approved these changes Jan 18, 2020
@prylabs-bulldozer prylabs-bulldozer bot merged commit 3e7e447 into master Jan 19, 2020
@delete-merged-branch delete-merged-branch bot deleted the makeAsync branch January 19, 2020 01:37
cryptomental pushed a commit to cryptomental/prysm that referenced this pull request Feb 24, 2020
* make rpc status requests async
* make whole block async
* fix nogo
* Update beacon-chain/sync/rpc_status.go
* Merge branch 'master' into makeAsync
* Merge refs/heads/master into makeAsync
cryptomental pushed a commit to cryptomental/prysm that referenced this pull request Feb 28, 2020
* make rpc status requests async
* make whole block async
* fix nogo
* Update beacon-chain/sync/rpc_status.go
* Merge branch 'master' into makeAsync
* Merge refs/heads/master into makeAsync
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

3 participants