Skip to content

[fix] use size in network fee voter keys #check-lint-warning

akrokr requested to merge akh/net_fee_voter into develop

Includes the TransactionSize field in the kv store key/record for network fee voters.

  • add TransactionSize field to ObservedNetworkFeeVoter
  • add ID() func to ObservedNetworkFeeVoter which includes size (will replace String())
  • add size param to GetObservedNetworkFeeVoter
  • if version > 128, store the size with the record and use ID() instead of String() for keying in GetObservedNetworkFeeVoter and SetObservedNetworkFeeVoter

#check-lint-warning is added since I'm updating a function signature used in archived code. the new param is only used after checking version within the function

Note: if any network fee voters are in the process of reaching consensus when the version upgrade is adopted, those fees will not reach consensus. I don't believe this to be an issue.

Edited by akrokr

Merge request reports