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

Shai Erera commented on LUCENE-4752:
------------------------------------

Adrien, I think that the change to LTC is a bit of an overkill. It doesn't test 
any new functionality except that you can sub-class SegmentMerger. Yet it did 
cause you to track down tests that might by affected b/c e.g. they are already 
slow tests and it will make them slower. Therefore I think that we should test 
this extensibility in a separate, dedicated test, which e.g. sub-classes SM and 
say reorders the list of atomic readers, so that we're sure this sub-classing 
not only compiled, but actually did something different than if we didn't 
sub-class.

Also, Robert made this comment:

bq. SegmentWriteState's fieldinfos should be final. Some of these little things 
took a long time to accomplish to give us a great deal of safety inside lucene 
internally.

Is it possible to make fieldInfos final? Since sorting segments has nothing to 
do with field infos (all that we need is to wrap the readers with SortingAR), 
we should only modify what's absolutely necessary to accomplish that.
                
> Merge segments to sort them
> ---------------------------
>
>                 Key: LUCENE-4752
>                 URL: https://issues.apache.org/jira/browse/LUCENE-4752
>             Project: Lucene - Core
>          Issue Type: New Feature
>          Components: core/index
>            Reporter: David Smiley
>            Assignee: Adrien Grand
>         Attachments: LUCENE-4752.patch
>
>
> It would be awesome if Lucene could write the documents out in a segment 
> based on a configurable order.  This of course applies to merging segments 
> to. The benefit is increased locality on disk of documents that are likely to 
> be accessed together.  This often applies to documents near each other in 
> time, but also spatially.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to