lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Michael McCandless (Commented) (JIRA)" <>
Subject [jira] [Commented] (LUCENE-3778) Create a grouping convenience class
Date Thu, 16 Feb 2012 18:48:59 GMT


Michael McCandless commented on LUCENE-3778:

bq. Would you also handle block (single pass) grouping with the same class...?

I think we can do this. The block grouping returns TopGroups as result.


bq. I guess you'd then .getAllGroups(), .getAllGroupHeads() after .search(...)?

Yes, we need that. In the case of getAllGroups() the TopGroups#totalGroupCount field can be
used when the user is only interested in the number of matching groups.


bq. Hmm would we try to handle Term/BytesRef and Function/MutableValue with the same class?

With generics?

I think so... but I think it may get tricky.  Like, I think you should
specify up front (to GroupingSearch ctor) the required things about
your request (block join OR group field OR field + DV type OR function
VS/ctx map), setters for the numerous optional things (sort,
groupSort, getScores, getMaxScores, maxDocsPerGroup) and maybe params
to search for the per-requesty things (topNGroups, groupOffset,

But then the <T> will depend on which ctor you used...?  Not sure how
it'd work...

bq. Maybe distributed grouping needs its own class? Since the usage is different from a non
distributed grouping.


Maybe we can do this for join module too!

> Create a grouping convenience class
> -----------------------------------
>                 Key: LUCENE-3778
>                 URL:
>             Project: Lucene - Java
>          Issue Type: Improvement
>          Components: modules/grouping
>            Reporter: Martijn van Groningen
> Currently the grouping module has many collector classes with a lot of different options
per class. I think it would be a good idea to have a GroupUtil (Or another name?) convenience
class. I think this could be a builder, because of the many options (sort,sortWithinGroup,groupOffset,groupCount
and more) and implementations (term/dv/function) grouping has.

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