Lucene - Core
  1. Lucene - Core
  2. LUCENE-4893

Facet counts in FacetsAccumulator.facetArrays are multiplied as many times as FacetsCollector.getFacetResults is called.

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 4.2
    • Fix Version/s: 4.3, Trunk
    • Component/s: modules/facet
    • Labels:
      None
    • Lucene Fields:
      New, Patch Available

      Description

      In lucene 4.1, only StandardFacetsAccumulator could be instantiated.
      And as of lucene 4.2, it became possible to instantiate FacetsAccumulator.

      I invoked FacetsCollector.getFacetResults twice, and I saw doubled facet counts.
      If I invoke it three times, I see facet counts multiplied three times.
      It all happens in FacetsAccumulator.accumulate.

      StandardFacetsAccumulator doesn't have this bug since it frees facetArrays whenever StandardFacetsAccumulator.accumulate is called.

      Is it a feature or a bug?

      1. LUCENE-4893.patch
        7 kB
        Shai Erera
      2. LUCENE-4893.patch
        10 kB
        Shai Erera
      3. LUCENE-4893.patch
        7 kB
        Shai Erera
      4. LUCENE-4893.patch
        6 kB
        Shai Erera

        Activity

        Uwe Schindler made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Shai Erera made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Lucene Fields New [ 10121 ] New,Patch Available [ 10121, 10120 ]
        Assignee Shai Erera [ shaie ]
        Fix Version/s 5.0 [ 12321663 ]
        Fix Version/s 4.3 [ 12324143 ]
        Resolution Fixed [ 1 ]
        Shai Erera made changes -
        Attachment LUCENE-4893.patch [ 12576300 ]
        Shai Erera made changes -
        Attachment LUCENE-4893.patch [ 12576285 ]
        Shai Erera made changes -
        Attachment LUCENE-4893.patch [ 12576282 ]
        Shai Erera made changes -
        Attachment LUCENE-4893.patch [ 12576273 ]
        crocket made changes -
        Description In lucene 4.1, only StandardFacetsAccumulator could be instantiated.
        And as of lucene 4.2, it became possible to instantiate FacetsAccumulator.

        I invoked FacetCollector.getFacetResults twice, and I saw doubled facet counts.
        If I invoke it three times, I see facet counts multiplied three times.
        It all happens in FacetsAccumulator.accumulate.

        StandardFacetsAccumulator doesn't have this bug since it frees facetArrays whenever StandardFacetsAccumulator.accumulate is called.

        Is it a feature or a bug?
        In lucene 4.1, only StandardFacetsAccumulator could be instantiated.
        And as of lucene 4.2, it became possible to instantiate FacetsAccumulator.

        I invoked FacetsCollector.getFacetResults twice, and I saw doubled facet counts.
        If I invoke it three times, I see facet counts multiplied three times.
        It all happens in FacetsAccumulator.accumulate.

        StandardFacetsAccumulator doesn't have this bug since it frees facetArrays whenever StandardFacetsAccumulator.accumulate is called.

        Is it a feature or a bug?
        crocket made changes -
        Summary Facet counts in FacetsAccumulator.facetArrays are multiplied as many times as FacetCollector.getFacetResults is called. Facet counts in FacetsAccumulator.facetArrays are multiplied as many times as FacetsCollector.getFacetResults is called.
        crocket made changes -
        Description In lucene 4.1, only StandardFacetsAccumulator could be instantiated.
        And as of lucene 4.2, it became possible to instantiate FacetsAccumulator.

        I invoked FacetCollector.getFacetResults twice, and I saw doubled facet counts.

        If I invoke it three times, I see facet counts multiplied three times.

        It all happens in FacetsAccumulator.accumulate.

        StandardFacetsAccumulator doesn't have this bug since it frees facetArrays whenever StandardFacetsAccumulator.accumulate is called.

        Is it a feature or a bug?
        In lucene 4.1, only StandardFacetsAccumulator could be instantiated.
        And as of lucene 4.2, it became possible to instantiate FacetsAccumulator.

        I invoked FacetCollector.getFacetResults twice, and I saw doubled facet counts.
        If I invoke it three times, I see facet counts multiplied three times.
        It all happens in FacetsAccumulator.accumulate.

        StandardFacetsAccumulator doesn't have this bug since it frees facetArrays whenever StandardFacetsAccumulator.accumulate is called.

        Is it a feature or a bug?
        crocket made changes -
        Field Original Value New Value
        Description In lucene 4.1, only StandardFacetsAccumulator was instantiable.
        And FacetsAccumulator became instantiable as of lucene 4.2

        I invoked FacetCollector.getFacetResults twice, and I saw doubled facet counts.

        If I invoke it three times, I see facet counts multiplied three times.

        It all happens in FacetsAccumulator.accumulate.

        StandardFacetsAccumulator doesn't have this bug since it frees facetArrays whenever StandardFacetsAccumulator.accumulate is called.

        Is it a feature or a bug?
        In lucene 4.1, only StandardFacetsAccumulator could be instantiated.
        And as of lucene 4.2, it became possible to instantiate FacetsAccumulator.

        I invoked FacetCollector.getFacetResults twice, and I saw doubled facet counts.

        If I invoke it three times, I see facet counts multiplied three times.

        It all happens in FacetsAccumulator.accumulate.

        StandardFacetsAccumulator doesn't have this bug since it frees facetArrays whenever StandardFacetsAccumulator.accumulate is called.

        Is it a feature or a bug?
        crocket created issue -

          People

          • Assignee:
            Shai Erera
            Reporter:
            crocket
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development