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

Reply via email to