lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Robert Muir (JIRA)" <>
Subject [jira] [Updated] (LUCENE-5493) Rename Sorter, NumericDocValuesSorter, and fix javadocs
Date Thu, 06 Mar 2014 12:38:43 GMT


Robert Muir updated LUCENE-5493:

    Attachment: LUCENE-5493-poc.patch

Here is a very simple proof of concept patch.

I made a SortSorter (extends existing Sorter api and takes o.a.l.s.Sort), removed NumericDocValuesSorter,
and replaced it with this more general Sort-Sorter in all tests and they pass.

So my next step would be to remove public apis like Sorter/DocMap and make that all internal.
SortingMP and EarlyTerminatingSortingCollector would just take Sort directly.

BlockJoinSorter needs to be cutover to a regular comparator. And in suggest/ there is a custom
comparator... that i think doesnt need to be custom and is just sorting on a dv field.

> Rename Sorter, NumericDocValuesSorter, and fix javadocs
> -------------------------------------------------------
>                 Key: LUCENE-5493
>                 URL:
>             Project: Lucene - Core
>          Issue Type: Bug
>            Reporter: Robert Muir
>            Assignee: Robert Muir
>         Attachments: LUCENE-5493-poc.patch
> Its not clear to users that these are for this super-expert thing of pre-sorting the
index. From the names and documentation they think they should use them instead of Sort/SortField.
> These need to be renamed or, even better, the API fixed so they aren't public classes.

This message was sent by Atlassian JIRA

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

View raw message