v3.0: Sort in **descending** stake order (backport of #8787)#8805
v3.0: Sort in **descending** stake order (backport of #8787)#8805AshwinSekar merged 1 commit intov3.0from
Conversation
(cherry picked from commit cefa44b)
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## v3.0 #8805 +/- ##
=======================================
Coverage 83.4% 83.4%
=======================================
Files 810 810
Lines 366213 366213
=======================================
+ Hits 305677 305701 +24
+ Misses 60536 60512 -24 🚀 New features to boost your workflow:
|
|
out of an abundance of caution, we want to see this on testnet for a bit before accepting this one |
|
@AshwinSekar @OliverNChalk - can we check metrics from testnet? |
|
it at least doesn't seem to be adversely affected. Still a minority of stake on v3.1 (~25%), so wouldn't expect differences to be too pronounced |
doesn't this affect block inclusion so we should see higher staked votes landing earlier in upgraded nodes? |
|
Yeah, I think we're expecting confirmation lag to trend downwards which it appears to (marginally). |
In theory, yes, but spot checking a few slots from I'm assuming we're sorting properly within vote storage, but latency into the scheduler likely still dominates as vote storage remains small (not sure if we have a metric that can confirm this) |
|
how does the trend look this week. should be more adoption by now |
t-nelson
left a comment
There was a problem hiding this comment.
received offline confirmation that this is in fact effective


Problem
Summary of Changes
Fixes #
This is an automatic backport of pull request #8787 done by [Mergify](https://mergify.com).