Skip to content

Commit

Permalink
fix: fix discontinue panic issue (#330)
Browse files Browse the repository at this point in the history
  • Loading branch information
forcodedancing committed Jul 7, 2023
1 parent c64374d commit c860b90
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions x/storage/abci.go
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,8 @@ func EndBlocker(ctx sdk.Context, keeper k.Keeper) {
// delete objects
deleted, err := keeper.DeleteDiscontinueObjectsUntil(ctx, blockTime, deletionMax)
if err != nil {
panic("fail to delete objects, err " + err.Error())
ctx.Logger().Error("fail to delete objects", "err", err.Error())
//panic("fail to delete objects, err " + err.Error())
}

if deleted >= deletionMax {
Expand All @@ -38,7 +39,8 @@ func EndBlocker(ctx sdk.Context, keeper k.Keeper) {
// delete buckets
_, err = keeper.DeleteDiscontinueBucketsUntil(ctx, blockTime, deletionMax-deleted)
if err != nil {
panic("fail to delete buckets, err " + err.Error())
ctx.Logger().Error("fail to delete buckets", "err", err.Error())
//panic("fail to delete buckets, err " + err.Error())
}
keeper.PersistDeleteInfo(ctx)

Expand Down

0 comments on commit c860b90

Please sign in to comment.