[jira] [Comment Edited] (SOLR-5079) Create ngroups for pivot faceting

["Sandro Mario Zbinden (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/SOLR-5079?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13720857#comment-13720857 ] 

Sandro Mario Zbinden edited comment on SOLR-5079 at 7/26/13 3:16 PM:
---------------------------------------------------------------------

I created a first patch that implement this feature and reduce the traffic for loading facet pivoting. In this patch there is a disadvantage. If facet.limit is not set to -1 the ngroup count is affected.

The main change of the code is that i added the ngroup in the PivotFacetHelper class

{code:title=PivotFacetHelper.java|borderStyle=solid}
          boolean ngroups = params.getFieldBool(field, FacetParams.FACET_PIVOT_NGROUP, false);
          if (ngroups) {
            int ngroup = 0;
            for (Entry<String,Integer> entry  : nl) {
              if (entry.getValue() > 0) {
                ngroup = ngroup + 1;
              }
            }
            pivot.add("ngroup", ngroup);  
          }
{code}
                
      was (Author: sandrozbinden):
    I created a first patch that implement this featuer and reduce the traffic for loading facet pivoting. In this patch there is a disadvantage. If facet.limit is not set to -1 the ngroup count is affected.

                  
> Create ngroups for pivot faceting
> ---------------------------------
>
>                 Key: SOLR-5079
>                 URL: https://issues.apache.org/jira/browse/SOLR-5079
>             Project: Solr
>          Issue Type: Improvement
>    Affects Versions: 5.0, 4.5
>            Reporter: Sandro Mario Zbinden
>              Labels: facet, pivot
>         Attachments: SOLR-5079.patch
>
>   Original Estimate: 24h
>  Remaining Estimate: 24h
>
> To save network traffic it would be great to now how many entries a facet list contains without loading the complete facet list. This issue is created because of an out of memory in loading the pivot facet.
> The facet.pivot result would then look like
> q=facet.pivot=cat,id&facet.pivot.ngroup=true
> <arr name="cat,id">
>  <lst>
>    <str name="field">cat/str>
>    <str name="value">a</str>
>    <int name="count">20</int>
>    <arr name="pivot">
>    <lst>
>      <str name="field">id</str>
>      <int name="value">69</int>
>      <int name="count">10</int>
>    </lst>
>    <lst>
>      <str name="field">id</str>
>      <int name="value">71</int>
>      <int name="count">10</int>
>    </lst>
>    <int name="ngroup">2</int> <-- The new ngroup parm
>  </lst>
> </arr>
> If you add another new param for example facet.pivot.visible the
> result could create less traffic
> especially if there are a lot of ids and the param facet.limit=-1 is set
> q=facet.pivot=cat,id&facet.ngroup=true&f.id.facet.pivot.visible=false
> <arr name="cat,id">
>  <lst>
>    <str name="field">cat/str>
>    <str name="value">a</str>
>    <int name="count">20</int>
>    <int name="ngroup">2</int> <-- The new ngroup parm
>  </lst>
> </arr>

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org