lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Martijn van Groningen (Closed) (JIRA)" <>
Subject [jira] [Closed] (LUCENE-3482) Refactor grouping module to be more maintainable
Date Wed, 02 Nov 2011 07:39:32 GMT


Martijn van Groningen closed LUCENE-3482.

    Resolution: Won't Fix

Ok I think this issue should be closed. The trade off for more maintainable code has a negative
impact on performance that is just a bit too much.  
> Refactor grouping module to be more maintainable
> ------------------------------------------------
>                 Key: LUCENE-3482
>                 URL:
>             Project: Lucene - Java
>          Issue Type: Improvement
>          Components: modules/grouping
>    Affects Versions: 4.0
>            Reporter: Martijn van Groningen
>             Fix For: 4.0
>         Attachments: LUCENE-3482.patch
> Currently we have 4 types of grouping collectors and 8 concrete subclasses in Lucene
/ Solr. In current architecture for each type of collector two concrete subclasses need to
be created. An implementation optimized for single term based groups and a more general implementation
that works with MutableValue to also support grouping by functions. If we want for example
group by IndexDocValues each type of grouping collector needs to have three concrete subclasses.
This design isn't very maintainable.
> I think it is best to introduce a concept that knows how deals with dealing groups for
all the different sources. Therefore the grouping module should depend on the queries module,
so that grouping can reuse the ValueSource concept. A term based concrete impl. of this concept
knows for example to use the DocValues.ord() method. Or more generic concrete impl. will use

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:!default.jspa
For more information on JIRA, see:


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

View raw message