Does "uniq" expression sounds good to use for UniqueMetric class?
Thanks, Susheel On Tue, Jul 4, 2017 at 5:45 PM, Susheel Kumar <susheel2...@gmail.com> wrote: > Hello Joel, > > I tried to create a patch to add UniqueMetric and it works, but soon > realized, we have UniqueStream as well and can't load both of them (like > below) when required, since both uses "unique" keyword. > > Any advice how we can handle this. Come up with different keyword for > UniqueMetric or rename UniqueStream etc..? > > StreamFactory factory = new StreamFactory() > .withCollectionZkHost (...) > .withFunctionName("facet", FacetStream.class) > .withFunctionName("sum", SumMetric.class) > .withFunctionName("unique", UniqueStream.class) > .withFunctionName("unique", UniqueMetric.class) > > On Thu, Jun 29, 2017 at 9:32 AM, Joel Bernstein <joels...@gmail.com> > wrote: > >> This is mainly due to focus on other things. It would great to support all >> the aggregate functions in facet, rollup and timeseries expressions. >> >> Joel Bernstein >> http://joelsolr.blogspot.com/ >> >> On Thu, Jun 29, 2017 at 8:23 AM, Zheng Lin Edwin Yeo < >> edwinye...@gmail.com> >> wrote: >> >> > Hi, >> > >> > We are working on the Solr Streaming expression, using the facet stream >> > source. >> > >> > As the underlying structure is using JSON Facet, would like to find out >> why >> > the unique() metrics is not supported? Currently, it only supports >> sum(col) >> > , avg(col), min(col), max(col), count(*) >> > >> > I'm using Solr 6.5.1 >> > >> > Regards, >> > Edwin >> > >> > >