Details
-
Bug
-
Resolution: Fixed
-
Critical
-
Cheshire-Cat
-
Triaged
-
1
-
Unknown
-
KV 2021-Oct-21
Description
- Most CheckpointConfig members store configuration dynamic parameters in non-atomic types
- We need to remove hard-coded default/min/max values and rely on configuration
- Once hard-coded values are removed, also most of the validation code can be removed and we can just rely on the built-in validation fixture in configuration
- It would be nice to store the EPEngine& (ref to the owning EPEngine) in CheckpointConfig - that would allow to remove some redundant/test-only ctor. For doing that we need to move some existing checkpoint component-test-class to any Engine test class
Attachments
For Gerrit Dashboard: MB-48529 | ||||||
---|---|---|---|---|---|---|
# | Subject | Branch | Project | Status | CR | V |
163861,6 | MB-48529: Remove CheckpointConfig::addConfigChangeListener | master | kv_engine | Status: NEW | -1 | -1 |