[ 
https://issues.apache.org/jira/browse/LUCENE-7299?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15298325#comment-15298325
 ] 

David Smiley commented on LUCENE-7299:
--------------------------------------

Cool indeed!  Might you give us some advise on when to use this sort 
implementation?  The [Wikipedia page on radix 
sort|https://en.wikipedia.org/wiki/Radix_sort] is an interesting read.  So this 
is just for integers.  Some more javadocs and implementation comments within 
LSBRadixSorter would be helpful.  Would it make sense for an implementation to 
extend o.a.l.u.Sorter?

> BytesRefHash.sort() should use radix sort?
> ------------------------------------------
>
>                 Key: LUCENE-7299
>                 URL: https://issues.apache.org/jira/browse/LUCENE-7299
>             Project: Lucene - Core
>          Issue Type: Improvement
>            Reporter: Adrien Grand
>            Assignee: Adrien Grand
>            Priority: Minor
>         Attachments: LUCENE-7299.patch
>
>
> Switching DocIdSetBuilder to radix sort helped make things significantly 
> faster. We should be able to do the same with BytesRefHash.sort()?



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to