Remove the update lock on upgrade iterator (paritytech#6179)

frame/vesting/src/lib.rs

@@ -53,7 +53,7 @@ use codec::{Encode, Decode};
 use sp_runtime::{DispatchResult, RuntimeDebug, traits::{
 	StaticLookup, Zero, AtLeast32Bit, MaybeSerializeDeserialize, Convert
 }};
-use frame_support::{decl_module, decl_event, decl_storage, decl_error, ensure, IterableStorageMap};
+use frame_support::{decl_module, decl_event, decl_storage, decl_error, ensure};
 use frame_support::traits::{
 	Currency, LockableCurrency, VestingSchedule, WithdrawReason, LockIdentifier,
 	ExistenceRequirement, Get
@@ -181,13 +181,6 @@ decl_module! {
 
 		fn deposit_event() = default;
 
-		fn on_runtime_upgrade() -> frame_support::dispatch::Weight {
-			for (a, _) in Vesting::<T>::iter() {
-				let _ = Self::update_lock(a);
-			}
-			1_000_000_000_000
-		}
-
 		/// Unlock any vested funds of the sender account.
 		///
 		/// The dispatch origin for this call must be _Signed_ and the sender must have funds still