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

Shahar Davidson commented on SOLR-2894:
---------------------------------------

Hi Chris,

Thanks for the updated patch.

I started testing this latest patch and encountered a few problems when NULL 
values are present:
(1) If, for example, FIELD_A contains null values and FIELD_B does not, then 
"facet.pivot=FIELD_A,FIELD_B" will return more than 1 entry of NULLs for 
FIELD_A. To be exact, it return an NULL entry per shard. (In non-distributed 
search, there is a single NULL entry for FIELD_A)
(2) If facet.pivot=FIELD_B,FIELD_A then one may see that for a given of field 
FIELD_B there is more than 1 entry of FIELD_A with NULL value. (In 
non-distributed search, there's only 1 entry of a NULL value under a given 
FIELD_B)

It seems as if there's a problem when merging null values from cross-shard 
pivots.
Does the relevant test-case include some sort of check on data with null values?

As far as sorting is concerned, results seem to be sorted properly (per field 
as well).

Shahar.
                
> Implement distributed pivot faceting
> ------------------------------------
>
>                 Key: SOLR-2894
>                 URL: https://issues.apache.org/jira/browse/SOLR-2894
>             Project: Solr
>          Issue Type: Improvement
>            Reporter: Erik Hatcher
>             Fix For: 4.2, 5.0
>
>         Attachments: SOLR-2894.patch, SOLR-2894.patch, SOLR-2894.patch, 
> SOLR-2894.patch, SOLR-2894.patch, SOLR-2894-reworked.patch
>
>
> Following up on SOLR-792, pivot faceting currently only supports 
> undistributed mode.  Distributed pivot faceting needs to be implemented.

--
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: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to