Details
-
Improvement
-
Resolution: Fixed
-
Major - P3
-
None
-
None
-
None
-
Storage Execution NAMER
-
Fully Compatible
-
Execution NAMR Team 2023-09-18
Description
The time-series buckets have a control.version field that is currently either a 1 or 2 value. It is difficult to comprehend what the versions mean as integers as opposed to the descriptive names in timeseries_constants.h:
- kTimeseriesControlUncompressedVersion
- kTimeseriesControlCompressedVersion
Replace all magic numbers with these descriptive names.
A few examples:
- bucket_unpacker.h
- validate_adaptor.cpp
- bucket_unpacker_test.cpp