clintropolis opened a new issue #6823: Use an enum for aggregator cache key byte identifiers instead of static values URL: https://github.com/apache/incubator-druid/issues/6823 [Based on this discussion](https://github.com/apache/incubator-druid/pull/6397#discussion_r228986335) the idea is to use an enum instead of a pile of static variables for associating aggregators to cache keys in `AggregatorUtil`, and use a static initializer to ensure that it is impossible to define the same byte identifier twice. `CacheKeyBuilder` should also be modified to accept the enum instead of the byte value.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
