There are a number of functions that are currently being held up because of conflicting duplicate function names. We haven't come to an agreement yet on the best way forward for this yet. I think we should open a separate ticket to discuss how best to handle this issue.
Joel Bernstein http://joelsolr.blogspot.com/ On Wed, Jul 5, 2017 at 10:04 AM, Susheel Kumar <susheel2...@gmail.com> wrote: > 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 > >> > > >> > > > > >
