Re: Add support in FacetsComponent for facet.method=uif

2016-01-03 Thread Jamie Johnson
The patch adds facet.method=uif and then delegates all of the work to the JSON Faceting API to do the work. I had originally added a facet.method=dv and made the original facet.method=fc work using the UnInvertedField but wanted to avoid making a change that would introduce unexpected behavior.

Re: Add support in FacetsComponent for facet.method=uif

2016-01-03 Thread Jamie Johnson
Are you looking at 8466 or 8096? The patch on 8466 is the one I'm referencing. I should remove the other as it is more change than I think should be done for this ticket. Jamie On Jan 3, 2016 8:47 PM, "William Bell" wrote: > Ok the path appears to have dv and uif in

Re: Add support in FacetsComponent for facet.method=uif

2016-01-03 Thread William Bell
Interesting that facet.method=dv or facet.method=uif. What is the difference? On Sun, Jan 3, 2016 at 6:44 AM, Jamie Johnson wrote: > For those interested I created a separate jira issue for this but forgot to > attach earlier. > >

Re: Add support in FacetsComponent for facet.method=uif

2016-01-03 Thread William Bell
Ok the path appears to have dv and uif in there.? On Sun, Jan 3, 2016 at 4:40 PM, Jamie Johnson wrote: > The patch adds facet.method=uif and then delegates all of the work to the > JSON Faceting API to do the work. I had originally added a facet.method=dv > and made the

Re: Add support in FacetsComponent for facet.method=uif

2016-01-03 Thread Jamie Johnson
For those interested I created a separate jira issue for this but forgot to attach earlier. https://issues.apache.org/jira/browse/SOLR-8466 On Jan 2, 2016 8:45 PM, "William Bell" wrote: > Yes we would like backward compatibility. We cannot switch all the facet > fields to

Re: Add support in FacetsComponent for facet.method=uif

2016-01-02 Thread William Bell
Yes we would like backward compatibility. We cannot switch all the facet fields to DocValues and our faceting is slow. Please... On Fri, Jan 1, 2016 at 7:41 AM, Jamie Johnson wrote: > Is there any interest in this? While i think it's important and inline > with faceting

Re: Add support in FacetsComponent for facet.method=uif

2016-01-01 Thread Jamie Johnson
Is there any interest in this? While i think it's important and inline with faceting available in the new json facet api, I've seen no discussion on it so I'm wondering if it's best I add support for this using a custom facet component even though the majority of the component will be a copy

Add support in FacetsComponent for facet.method=uif

2015-12-22 Thread Jamie Johnson
I had previously piggybacked on another post, but I think it may have been lost there. I had a need to do UnInvertedField based faceting in the FacetsComponent and as such started looking at what would be required to implement something similar to what the JSON Facets based API does in this