You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2019/01/30 09:56:47 UTC
[GitHub] clintropolis commented on issue #6960: standardize naming of
'no-op' aggregators
clintropolis commented on issue #6960: standardize naming of 'no-op' aggregators
URL: https://github.com/apache/incubator-druid/pull/6960#issuecomment-458882776
Since I'm renaming a few files in the area already, any upvotes for renaming some of the other datasketches extension post-aggregator types to make navigating to the right file easier? I would like to switch around what is currently the middle of the compound name to the front, like `ArrayOfDoublesSketchToNumEntriesPostAggregator` -> `NumEntriesArrayOfDoublesSketchPostAggregator`
<img width="393" alt="screen shot 2019-01-30 at 1 17 41 am" src="https://user-images.githubusercontent.com/1577461/51972956-7cf5ed00-2431-11e9-8073-052662b86c9b.png">
(Open to other ideas too)
----------------------------------------------------------------
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:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org