Details

    • Type: New Feature New Feature
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.0-ALPHA
    • Component/s: modules/grouping
    • Labels:
      None

      Description

      Support a second pass collector that counts unique field values of a field per group.
      This is just one example of group statistics that one might want.

      1. LUCENE-3444.patch
        50 kB
        Martijn van Groningen
      2. LUCENE-3444.patch
        41 kB
        Martijn van Groningen
      3. LUCENE-3444.patch
        16 kB
        Martijn van Groningen
      4. LUCENE-3444.patch
        14 kB
        Martijn van Groningen

        Activity

        Hide
        Martijn van Groningen added a comment - - edited

        Attached initial version of a second pass collector that counts the unique field values per group for a specific field.

        Show
        Martijn van Groningen added a comment - - edited Attached initial version of a second pass collector that counts the unique field values per group for a specific field.
        Hide
        Martijn van Groningen added a comment -

        Updated patch. I've split the DistinctCountCollector into abstract base class and a term based implementation. This allows other implementations such as IDV and function based implementations.

        Show
        Martijn van Groningen added a comment - Updated patch. I've split the DistinctCountCollector into abstract base class and a term based implementation. This allows other implementations such as IDV and function based implementations.
        Hide
        Martijn van Groningen added a comment -

        Updated the patch and added a docvalues based implementation.

        Things to do:

        • Add implementation that uses MutableValue.
        • Add random tests.
        Show
        Martijn van Groningen added a comment - Updated the patch and added a docvalues based implementation. Things to do: Add implementation that uses MutableValue. Add random tests.
        Hide
        Martijn van Groningen added a comment -

        Added new patch.

        • Updated patch to current trunk.
        • Added random test. Fails now in some cases.
        • Added function (mutable value) based implementation.

        It is almost ready to be committed!

        Show
        Martijn van Groningen added a comment - Added new patch. Updated patch to current trunk. Added random test. Fails now in some cases. Added function (mutable value) based implementation. It is almost ready to be committed!
        Hide
        Martijn van Groningen added a comment -

        Committed to trunk.

        Show
        Martijn van Groningen added a comment - Committed to trunk.

          People

          • Assignee:
            Unassigned
            Reporter:
            Martijn van Groningen
          • Votes:
            1 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development