Closed nflaig closed 1 month ago
Attention: Patch coverage is 96.77419%
with 1 line
in your changes missing coverage. Please review.
Project coverage is 48.98%. Comparing base (
ad4ec77
) to head (e595ab7
). Report is 7 commits behind head on unstable.
✔️ no performance regression detected
🚀🚀 Significant benchmark improvement detected
Benchmark suite | Current: 61743deb118bb047731cab01b8748e76c2c02140 | Previous: 77006ea0ce9ac043b71b7aaa07b9fae6f38f5250 | Ratio |
---|---|---|---|
getExpectedWithdrawals 250000 eb:0.95,eth1:0.1,we:0.05,wn:0,smpl:219 | 17.668 us/op | 60.633 us/op | 0.29 |
by benchmarkbot/action
:tada: This PR is included in v1.23.0 :tada:
Motivation
Description
Adds validator identities endpoint which is a much more lightweight alternative to retrieve the validator index from state compared to getStateValidators and also allows SSZ encoding of response.
There still needs to be another endpoint (see https://github.com/ethereum/beacon-APIs/pull/449) before we can make use of this in the validator client as we currently also print out the status of the validator (although this is purely informational) but other clients that only need the index can make use of this as is.
Keep as draft for now until spec PR is merged.
Closes https://github.com/ChainSafe/lodestar/issues/7105