-
Notifications
You must be signed in to change notification settings - Fork 78
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
app/eth2wrap: implement parallel multi beacon client #1012
Conversation
@@ -88,3 +88,13 @@ func (s structured) Stack() zap.Field { | |||
func (s structured) Unwrap() error { | |||
return s.err | |||
} | |||
|
|||
// Is returns true if err is equaled to this structured error. | |||
func (s structured) Is(err error) bool { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
found this issue with our error package, so fixed it
Codecov Report
@@ Coverage Diff @@
## main #1012 +/- ##
==========================================
- Coverage 54.16% 52.78% -1.39%
==========================================
Files 118 123 +5
Lines 13215 13661 +446
==========================================
+ Hits 7158 7211 +53
- Misses 5030 5419 +389
- Partials 1027 1031 +4
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
} | ||
|
||
func (m multi) Address() string { | ||
// TODO(corver): return "best" address. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i assume this will come in subsequent PRs or it will be taken as part of separate issue?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can do as separate issue I think
Implements a parallel eth2 client to improve beacon node latency by using the "fastest" response.
category: feature
ticket: #960