Skip to content

Commit

Permalink
cmd/geth: change to non-fatal error message when legacy receipt stora…
Browse files Browse the repository at this point in the history
…ge is not implemented (#24603)

* cmd/geth: only check for presence of legacy receipts if developer mode is not enabled

* cmd/geth: degrade log level

* cmd/geth: fix format

Co-authored-by: Gary Rong <garyrong0905@gmail.com>
  • Loading branch information
jwasinger and rjl493456442 authored Mar 29, 2022
1 parent 1027cb5 commit b502b6a
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions cmd/geth/config.go
Original file line number Diff line number Diff line change
Expand Up @@ -174,9 +174,8 @@ func makeFullNode(ctx *cli.Context) (*node.Node, ethapi.Backend) {
}
isLegacy, _, err := dbHasLegacyReceipts(eth.ChainDb(), firstIdx)
if err != nil {
utils.Fatalf("Failed to check db for legacy receipts: %v", err)
}
if isLegacy {
log.Error("Failed to check db for legacy receipts", "err", err)
} else if isLegacy {
log.Warn("Database has receipts with a legacy format. Please run `geth db freezer-migrate`.")
}
}
Expand Down

0 comments on commit b502b6a

Please sign in to comment.