[AHM] Fast unstake and Bags list #563
Conversation
Signed-off-by: Oliver Tale-Yazdi <[email protected]>
…y-ahm-fast-unstake
Signed-off-by: Oliver Tale-Yazdi <[email protected]>
Signed-off-by: Oliver Tale-Yazdi <[email protected]>
Signed-off-by: Oliver Tale-Yazdi <[email protected]>
Signed-off-by: Oliver Tale-Yazdi <[email protected]>
There was a problem hiding this comment.
Need to check if we have to re-generate these values.
There was a problem hiding this comment.
No proper review given from me for this file. But I will do it on request. May be we better consult with staking team on this, will take more time from me.
There was a problem hiding this comment.
No proper review given from me for this file. But I will do it on request. May be we better consult with staking team on this, will take more time from me.
| type RuntimeEvent = RuntimeEvent; | ||
| type Currency = Balances; | ||
| type BatchSize = frame_support::traits::ConstU32<16>; | ||
| type Deposit = frame_support::traits::ConstU128<{ UNITS }>; |
There was a problem hiding this comment.
might need to be updated, since should cost less on a parachain
| MigrationDone, | ||
| } | ||
|
|
||
| pub type MigrationStageFor<T> = MigrationStage<<T as frame_system::Config>::AccountId>; | ||
| pub type MigrationStageFor<T> = MigrationStage< |
There was a problem hiding this comment.
Do we have some non written convention when we use For or Of suffix? I wished to keep it consistent (=
I had a feeling that Gav used For for storage items and Of for types, but I am not sure.
| }); | ||
| }, | ||
| e => { | ||
| log::error!(target: LOG_TARGET, "Error while migrating fast unstake: {:?}", e); |
There was a problem hiding this comment.
actually no need for error, since defensive is below. just would be nice to pass the e for defensive as well
| break; | ||
| } | ||
| } | ||
| if messages.len() > 10_000 { |
There was a problem hiding this comment.
temporary measure or not? can you explain please
| messages: Vec<RcBagsListMessage<T>>, | ||
| ) -> Result<(), Error<T>> { | ||
| let (mut good, mut bad) = (0, 0); | ||
| log::info!("Integrating {} BagsListMessages", messages.len()); |
To be merged into the AHM dev branch.
There should be no user impact here - both pallets have their storage migrated exactly as is and the used times are always in Epoch.