Skip to content

Conversation

@kderme
Copy link
Contributor

@kderme kderme commented Apr 6, 2022

This pr

  • Updates dependencies to current node master
  • It uses the new reward events. This includes Incremental rewards, Delta rewards, Restrained rewards. Validation is still done using the TotalRewardEvent and mir and deposits are taken from the MirRewards and RetiredPool event.
  • BulkOperations are removed since rewards are provided incrementally. In order to fully remove them we also cherry-pick Introduce Stake Slices #1086
  • AdjustRewards are greatly simplified. Since we get the deregistered rewards through a new event, the composite query to find them is removed.
  • shashReward, stashMir is removed, since instant rewards are now inserted through their event at the epoch boundary.
  • IndexCache is replaced by Cache, which is a cherry-pick of Cache for tables that are reused frequently #1074. This cache better suits the inremental way that reward/stake are provided. This new cache also is used for pools, previous block and multiassets.
  • Improves logging and tests for rewards.

@kderme kderme force-pushed the kderme/update-deps-events branch 4 times, most recently from d75629a to 9153ea3 Compare April 13, 2022 17:37
@kderme kderme force-pushed the kderme/update-deps-events branch 6 times, most recently from b0994ca to bb68efb Compare April 19, 2022 21:10
@kderme kderme marked this pull request as ready for review April 19, 2022 21:12
@kderme kderme requested a review from erikd as a code owner April 19, 2022 21:12
@kderme kderme force-pushed the kderme/update-deps-events branch 6 times, most recently from a174ae1 to ddb91cc Compare April 26, 2022 18:55
@kderme kderme mentioned this pull request Apr 26, 2022
@kderme kderme force-pushed the kderme/update-deps-events branch 3 times, most recently from f28e8b1 to 4a54285 Compare April 26, 2022 19:48
@kderme kderme force-pushed the kderme/update-deps-events branch from 4a54285 to 4e71e43 Compare April 26, 2022 22:24
@erikd erikd force-pushed the kderme/update-deps-events branch from a47f9ba to 3cec0f7 Compare April 27, 2022 09:02
@kderme kderme force-pushed the kderme/update-deps-events branch 2 times, most recently from d6e49b7 to d3a5a54 Compare April 27, 2022 23:28
@erikd erikd force-pushed the kderme/update-deps-events branch from d3a5a54 to 77b0f30 Compare April 28, 2022 01:46
@erikd erikd force-pushed the kderme/update-deps-events branch from 77b0f30 to 52ebe3b Compare April 28, 2022 03:19
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.

3 participants