osmosis icon indicating copy to clipboard operation
osmosis copied to clipboard

feat(twap): pruning "time to keep records" is configured via parameter

Open p0mvn opened this issue 2 years ago • 2 comments

Background

In #2294 , we introduced twap pruning. Currently, the period for which the records are kept is hardcoded: https://github.com/osmosis-labs/osmosis/blob/62aa54e894b501a7e8ce8866c3f755a187a8429e/x/twap/listeners.go#L31-L32

Instead, this should be set via parameter.

The implementation should be similar to #2289

Acceptance Criteria

  • pruning "time to keep records" is configured via parameter
  • tests added to ensure that it is set and used correctly

p0mvn avatar Aug 11 '22 20:08 p0mvn

Hi @vuong177 are you working on this?

stackman27 avatar Aug 12 '22 23:08 stackman27

Yes. I'm waiting for p0mvn's PR to be merged.

vuong177 avatar Aug 14 '22 15:08 vuong177