Any time we discuss moving things into core Druid I would love to see a list of dependencies that comes with it.
On Wed, Oct 30, 2019, 6:08 PM Jihoon Son <jihoon...@apache.org> wrote: > +1 on moving too. > > On Mon, Oct 28, 2019 at 12:46 PM Fangjin Yang <fang...@imply.io> wrote: > > > +1 on moving datasketches to core > > > > On Mon, Oct 28, 2019 at 12:36 PM Chi Cao Minh <chi.caom...@imply.io> > > wrote: > > > > > To support range partitioning for native parallel batch indexing, I’m > > > considering moving DataSketches from extensions to core (see > > > https://github.com/apache/incubator-druid/issues/8769 < > > > https://github.com/apache/incubator-druid/issues/8769> for details). > > > Having DataSketches in core would also allow us to switch usages of > > > HyperLogLogCollector to the better HLL implementation available in > > > DataSketches. One drawback is that moving DataSketches to core will > > > possibly block the work to upgrade DataSketches to the latest version: > > > https://github.com/apache/incubator-druid/pull/8647 < > > > https://github.com/apache/incubator-druid/pull/8647>. > > > > > > Any other thoughts on the pros/cons? > > > > > > Thanks, > > > Chi > > >