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

Michael Gibney commented on SOLR-14467:
---------------------------------------

Hi Hoss, I've been looking into this (over at SOLR-13132), but I noticed that 
the test patch attached here only removes the {{TestCloudJSONFacetSKG.java}} 
file from master, and doesn't add any tests?

I also have a more general question about the semantics of {{allBuckets}}. 
>From the ref guide:
{quote}A boolean. If true, adds an “allBuckets” bucket to the response, 
representing the union of all of the buckets. For multi-valued fields, this is 
different than a bucket for all of the documents in the domain since a single 
document can belong to multiple buckets. Defaults to false.
{quote}
The ref guide docs for {{allBuckets}} seem ambiguous to me wrt whether 
{{allBuckets}} represents the union of all buckets across the full domain, or 
the union of all _returned_ buckets. For the latter, {{allBuckets}} stats would 
be inconsistent across different settings of {{limit}} for deferred stats, but 
consistent for non-deferred ({{collectAcc}}) stats. My initial intuition is 
that having stats for {{allBuckets}} represent the union across all buckets 
(whether returned or not) would be preferable, but I think that would mean 
preventing _any_ deferral of stats when {{allBuckets==true}}, which is not 
currently done. But I think that approach (if chosen) would be pretty 
straightforward: if {{allBuckets==true}}, instead of creating any 
{{otherAccs}}, simply add all accs to {{collectAcc}} using {{MultiAcc}}?

> inconsistent server errors combining relatedness() with allBuckets:true
> -----------------------------------------------------------------------
>
>                 Key: SOLR-14467
>                 URL: https://issues.apache.org/jira/browse/SOLR-14467
>             Project: Solr
>          Issue Type: Bug
>      Security Level: Public(Default Security Level. Issues are Public) 
>          Components: Facet Module
>            Reporter: Chris M. Hostetter
>            Priority: Major
>         Attachments: SOLR-14467_test.patch
>
>
> While working on randomized testing for SOLR-13132 i discovered a variety of 
> different ways that JSON Faceting's "allBuckets" option can fail when 
> combined with the "relatedness()" function.
> I haven't found a trivial way to manual reproduce this, but i have been able 
> to trigger the failures with a trivial patch to {{TestCloudJSONFacetSKG}} 
> which i will attach.
> Based on the nature of the failures it looks like it may have something to do 
> with multiple segments of different sizes, and or resizing the SlotAccs ?
> The relatedness() function doesn't have much (any?) existing tests in place 
> that leverage "allBuckets" so this is probably a bug that has always existed 
> -- it's possible it may be excessively cumbersome to fix and we might 
> nee/wnat to just document that incompatibility and add some code to try and 
> detect if the user combines these options and if so fail with a 400 error?



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org

Reply via email to