diff --git a/app/export.go b/app/export.go index 280164f..474a6a0 100644 --- a/app/export.go +++ b/app/export.go @@ -154,7 +154,6 @@ func (app *DnApp) prepForZeroHeightGenesis(ctx sdk.Context, jailAllowedAddrs []s // update bond intra-tx counters. store := ctx.KVStore(app.keys[stakingtypes.StoreKey]) iter := sdk.KVStoreReversePrefixIterator(store, stakingtypes.ValidatorsKey) - counter := int16(0) for ; iter.Valid(); iter.Next() { addr := sdk.ValAddress(iter.Key()[1:]) @@ -169,7 +168,9 @@ func (app *DnApp) prepForZeroHeightGenesis(ctx sdk.Context, jailAllowedAddrs []s } app.StakingKeeper.SetValidator(ctx, validator) - counter++ + if validator.IsJailed() { + app.StakingKeeper.DeleteValidatorByPowerIndex(ctx, validator) + } } iter.Close() diff --git a/x/oracle/keeper/asset.go b/x/oracle/keeper/asset.go index 22ab5fc..8e03f35 100644 --- a/x/oracle/keeper/asset.go +++ b/x/oracle/keeper/asset.go @@ -10,10 +10,6 @@ import ( // GetAsset returns an types.Asset if exists. func (k Keeper) GetAsset(ctx sdk.Context, assetCode dnTypes.AssetCode) *types.Asset { - if cachedValue, found := k.cache.assets[assetCode.String()]; found { - return &cachedValue - } - store := ctx.KVStore(k.storeKey) assetStore := prefix.NewStore(store, types.AssetsPrefix) key, _ := assetCode.Marshal() @@ -26,8 +22,6 @@ func (k Keeper) GetAsset(ctx sdk.Context, assetCode dnTypes.AssetCode) *types.As asset := &types.Asset{} k.cdc.MustUnmarshalBinaryBare(bz, asset) - k.cache.assets[asset.AssetCode.String()] = *asset - return asset } @@ -74,6 +68,4 @@ func (k Keeper) setAsset(ctx sdk.Context, asset types.Asset) { bz := k.cdc.MustMarshalBinaryBare(&asset) assetStore.Set(key, bz) - - k.cache.assets[asset.AssetCode.String()] = asset } diff --git a/x/oracle/keeper/keeper.go b/x/oracle/keeper/keeper.go index 3c68b8f..10e9095 100644 --- a/x/oracle/keeper/keeper.go +++ b/x/oracle/keeper/keeper.go @@ -17,14 +17,6 @@ type Keeper struct { storeKey sdk.StoreKey cdc codec.BinaryMarshaler paramSpace paramTypes.Subspace - // - cache *keeperCache -} - -// keeperCache optimized Gas usage for frequent PostPrice operations. -type keeperCache struct { - oracles map[string]types.Oracle // key: AccAddress - assets map[string]types.Asset // key: AssetCode } // IsNominee checks if nominee exist within keeper parameters and returns sdk wrapped error. @@ -57,9 +49,5 @@ func NewKeeper( cdc: cdc, storeKey: storeKey, paramSpace: paramSpace, - cache: &keeperCache{ - oracles: make(map[string]types.Oracle), - assets: make(map[string]types.Asset), - }, } } diff --git a/x/oracle/keeper/oracle.go b/x/oracle/keeper/oracle.go index 535302d..280bcb2 100644 --- a/x/oracle/keeper/oracle.go +++ b/x/oracle/keeper/oracle.go @@ -8,10 +8,6 @@ import ( // GetOracle returns an types.Oracle if exists. func (k Keeper) GetOracle(ctx sdk.Context, addr string) *types.Oracle { - if cachedValue, found := k.cache.oracles[addr]; found { - return &cachedValue - } - store := ctx.KVStore(k.storeKey) oracleStore := prefix.NewStore(store, types.OraclesPrefix) key := types.GetOraclesKey(addr) @@ -24,8 +20,6 @@ func (k Keeper) GetOracle(ctx sdk.Context, addr string) *types.Oracle { oracle := &types.Oracle{} k.cdc.MustUnmarshalBinaryBare(bz, oracle) - k.cache.oracles[addr] = *oracle - return oracle } @@ -60,10 +54,6 @@ func (k Keeper) SetOracle(ctx sdk.Context, msg types.MsgSetOracle) error { // hasOracle checks if types.Oracle with addr is registered. func (k Keeper) hasOracle(ctx sdk.Context, addr string) bool { - if _, found := k.cache.oracles[addr]; found { - return found - } - store := ctx.KVStore(k.storeKey) oracleStore := prefix.NewStore(store, types.OraclesPrefix) key := types.GetOraclesKey(addr) @@ -79,6 +69,4 @@ func (k Keeper) setOracle(ctx sdk.Context, oracle types.Oracle) { bz := k.cdc.MustMarshalBinaryBare(&oracle) oracleStore.Set(key, bz) - - k.cache.oracles[oracle.AccAddress] = oracle }