lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Shai Erera (JIRA)" <>
Subject [jira] [Commented] (LUCENE-5533) TaxonomyFacetSumIntAssociations overflows, unpredicted results
Date Mon, 17 Mar 2014 11:21:46 GMT


Shai Erera commented on LUCENE-5533:

bq. But maybe we should break out a LongTaxonomyFacets instead?

+1, and especially as it's only relevant to association-based faceting.

Rob, is this something you really hit or just a random code review? I agree when you sum integers
there's a risk of overflowing, but I'm afraid if we introduce LongTaxoFacets users might want
to use it "just in case". The risk is that a single ord will overflow, right?

I wonder if we should use a packed long buffer instead of a plain long[] ... that's optimization
though. First let's agree that this is something that needs fixing.

> TaxonomyFacetSumIntAssociations overflows, unpredicted results
> --------------------------------------------------------------
>                 Key: LUCENE-5533
>                 URL:
>             Project: Lucene - Core
>          Issue Type: Bug
>          Components: modules/facet
>    Affects Versions: 4.7
>            Reporter: Rob Audenaerde
> {{TaxonomyFacetSumIntAssociations}} extends {{IntTaxonomyFacets}} which uses a {{int[]}}
to store values. If you sum a lot of integers in the IntAssociatoins, the {{int}} will overflow.
> The easiest fix seems to change the {{value[]}} to {{long}}?

This message was sent by Atlassian JIRA

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message