[refactor] remove calling GetLowestVersion where possible

This change stops calling keeper.GetLowestActiveVersion() in all places except once (in manager.go). This expensive kvstore function is now only called a single time per block, instead of potentially multiple times per transaction and even per thornode API query.

Also, in an effort for uniformity, renamed keeper.Version() --> keeper.GetVersion() to match the similar function name in mgr.GetVersion()

Might be easiest to review this PR commit by commit.

Edited by Son of Odin

Merge request reports

Loading