[GitHub] edgan8 edited a comment on issue #6581: Moments Sketch custom aggregator

[GitBox <gi...@apache.org>]

edgan8 edited a comment on issue #6581: Moments Sketch custom aggregator
URL: https://github.com/apache/incubator-druid/pull/6581#issuecomment-461197020
 
 
   @leerho I completely agree, it seems the concern is about the recommendations we provide to users. Due to concerns about robustness, I don't think the moments sketch should be a recommended first choice in standard environments either. For users with extreme demands, my understanding was that putting an extension package in contrib was a convenient to location allow them to experiment and can work on updating the documentation to make that more clear. If the druid maintainers have different plans for contrib then I can move this back to an external repository, I don't have a stake. 
   
   @jon-wei please let me know your thoughts on the best place to put this package moving forward, I am also happy keeping it in my own repository if that is more convenient.
   
   @AlexanderSaydakov those features are not difficult to add if people find them important.

----------------------------------------------------------------
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