Lucene - Core
  1. Lucene - Core
  2. LUCENE-3856

Create docvalues based grouped facet collector

    Details

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

      Description

      Create docvalues based grouped facet collector. Currently only term based collectors have been implemented (LUCENE-3802).

      1. LUCENE-3856.patch
        42 kB
        Martijn van Groningen
      2. LUCENE-3856.patch
        32 kB
        Martijn van Groningen
      3. LUCENE-3856.patch
        32 kB
        Martijn van Groningen

        Activity

        Hide
        Martijn van Groningen added a comment -

        Initial patch containing a implementation for bytes sorted. Random test fails for now...

        I think bytes sorted is the only impl that makes since other implementations don't work with ordinals and thus would be much slower.

        Show
        Martijn van Groningen added a comment - Initial patch containing a implementation for bytes sorted. Random test fails for now... I think bytes sorted is the only impl that makes since other implementations don't work with ordinals and thus would be much slower.
        Hide
        Martijn van Groningen added a comment - - edited

        Updated patch. Random test passes now. Need to add some more jdocs and remove some duplicate code and then it should be ready.

        Show
        Martijn van Groningen added a comment - - edited Updated patch. Random test passes now. Need to add some more jdocs and remove some duplicate code and then it should be ready.
        Hide
        Martijn van Groningen added a comment -

        Some code refactorings and added some more jdocs.

        Show
        Martijn van Groningen added a comment - Some code refactorings and added some more jdocs.
        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:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development