Re: Add support in FacetsComponent for facet.method=uif
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 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 original facet.method=fc work using the UnInvertedField but > > wanted to avoid making a change that would introduce unexpected behavior. > > While I think it's strange that facet.method=dv does not exist and fc > > defaults to dv I think if we wanted to change that it should be done in > > another ticket. > > > > On Sun, Jan 3, 2016 at 4:18 PM, William Bell > wrote: > > > > > 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. > > > > > > > > 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 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 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 > > > > > > which is prefer to not need to maintain separately. > > > > > > > > > > > > Jamie > > > > > > On Dec 22, 2015 12:37 PM, "Jamie Johnson" > > wrote: > > > > > > > > > > > > > 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 > > > > > > > regard. The patch that I have in this regard works and is > > attached > > > > to > > > > > > > https://issues.apache.org/jira/browse/SOLR-8096, is that > > > appropriate > > > > > or > > > > > > > should I create a new ticket to specifically add this support? > > > > > > > > > > > > > > -Jamie > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > Bill Bell > > > > > billnb...@gmail.com > > > > > cell 720-256-8076 > > > > > > > > > > > > > > > > > > > > > -- > > > Bill Bell > > > billnb...@gmail.com > > > cell 720-256-8076 > > > > > > > > > -- > Bill Bell > billnb...@gmail.com > cell 720-256-8076 >
Re: Add support in FacetsComponent for facet.method=uif
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 original facet.method=fc work using the UnInvertedField but > wanted to avoid making a change that would introduce unexpected behavior. > While I think it's strange that facet.method=dv does not exist and fc > defaults to dv I think if we wanted to change that it should be done in > another ticket. > > On Sun, Jan 3, 2016 at 4:18 PM, William Bell wrote: > > > 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. > > > > > > 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 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 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 > > > > > which is prefer to not need to maintain separately. > > > > > > > > > > Jamie > > > > > On Dec 22, 2015 12:37 PM, "Jamie Johnson" > wrote: > > > > > > > > > > > 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 > > > > > > regard. The patch that I have in this regard works and is > attached > > > to > > > > > > https://issues.apache.org/jira/browse/SOLR-8096, is that > > appropriate > > > > or > > > > > > should I create a new ticket to specifically add this support? > > > > > > > > > > > > -Jamie > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > Bill Bell > > > > billnb...@gmail.com > > > > cell 720-256-8076 > > > > > > > > > > > > > > > -- > > Bill Bell > > billnb...@gmail.com > > cell 720-256-8076 > > > -- Bill Bell billnb...@gmail.com cell 720-256-8076
Re: Add support in FacetsComponent for facet.method=uif
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. While I think it's strange that facet.method=dv does not exist and fc defaults to dv I think if we wanted to change that it should be done in another ticket. On Sun, Jan 3, 2016 at 4:18 PM, William Bell wrote: > 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. > > > > 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 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 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 > > > > which is prefer to not need to maintain separately. > > > > > > > > Jamie > > > > On Dec 22, 2015 12:37 PM, "Jamie Johnson" wrote: > > > > > > > > > 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 > > > > > regard. The patch that I have in this regard works and is attached > > to > > > > > https://issues.apache.org/jira/browse/SOLR-8096, is that > appropriate > > > or > > > > > should I create a new ticket to specifically add this support? > > > > > > > > > > -Jamie > > > > > > > > > > > > > > > > > > > > > -- > > > Bill Bell > > > billnb...@gmail.com > > > cell 720-256-8076 > > > > > > > > > -- > Bill Bell > billnb...@gmail.com > cell 720-256-8076 >
Re: Add support in FacetsComponent for facet.method=uif
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. > > 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 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 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 > > > which is prefer to not need to maintain separately. > > > > > > Jamie > > > On Dec 22, 2015 12:37 PM, "Jamie Johnson" wrote: > > > > > > > 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 > > > > regard. The patch that I have in this regard works and is attached > to > > > > https://issues.apache.org/jira/browse/SOLR-8096, is that appropriate > > or > > > > should I create a new ticket to specifically add this support? > > > > > > > > -Jamie > > > > > > > > > > > > > > > -- > > Bill Bell > > billnb...@gmail.com > > cell 720-256-8076 > > > -- Bill Bell billnb...@gmail.com cell 720-256-8076
Re: Add support in FacetsComponent for facet.method=uif
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 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 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 > > which is prefer to not need to maintain separately. > > > > Jamie > > On Dec 22, 2015 12:37 PM, "Jamie Johnson" wrote: > > > > > 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 > > > regard. The patch that I have in this regard works and is attached to > > > https://issues.apache.org/jira/browse/SOLR-8096, is that appropriate > or > > > should I create a new ticket to specifically add this support? > > > > > > -Jamie > > > > > > > > > -- > Bill Bell > billnb...@gmail.com > cell 720-256-8076 >
Re: Add support in FacetsComponent for facet.method=uif
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 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 > which is prefer to not need to maintain separately. > > Jamie > On Dec 22, 2015 12:37 PM, "Jamie Johnson" wrote: > > > 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 > > regard. The patch that I have in this regard works and is attached to > > https://issues.apache.org/jira/browse/SOLR-8096, is that appropriate or > > should I create a new ticket to specifically add this support? > > > > -Jamie > > > -- Bill Bell billnb...@gmail.com cell 720-256-8076
Re: Add support in FacetsComponent for facet.method=uif
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 which is prefer to not need to maintain separately. Jamie On Dec 22, 2015 12:37 PM, "Jamie Johnson" wrote: > 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 > regard. The patch that I have in this regard works and is attached to > https://issues.apache.org/jira/browse/SOLR-8096, is that appropriate or > should I create a new ticket to specifically add this support? > > -Jamie >
Add support in FacetsComponent for facet.method=uif
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 regard. The patch that I have in this regard works and is attached to https://issues.apache.org/jira/browse/SOLR-8096, is that appropriate or should I create a new ticket to specifically add this support? -Jamie
