[
https://issues.apache.org/jira/browse/LUCENE-3586?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13167672#comment-13167672
]
Michael McCandless commented on LUCENE-3586:
--------------------------------------------
Hi Luca,
Actually I don't like all the generics required in CommandLineUtil solely to
support LTC's case; if not for LTC the util would be much simpler (just one
method taking a String class name/path and returning a Directory instance).
So.... I think we should go back, and leave LTC with its current (private) code
for finding the FS/Directory class/instance, and keep CommandLineUtil simple?
Sorry!
Otherwise the patch looks great! Can you add the lost " // for javadocs"
comment on one of CheckIndex's imports? Thanks.
> Choose a specific Directory implementation running the CheckIndex main
> ----------------------------------------------------------------------
>
> Key: LUCENE-3586
> URL: https://issues.apache.org/jira/browse/LUCENE-3586
> Project: Lucene - Java
> Issue Type: Improvement
> Reporter: Luca Cavanna
> Assignee: Luca Cavanna
> Priority: Minor
> Attachments: LUCENE-3586.patch, LUCENE-3586.patch, LUCENE-3586.patch
>
>
> It should be possible to choose a specific Directory implementation to use
> during the CheckIndex process when we run it from its main.
> What about an additional main parameter?
> In fact, I'm experiencing some problems with MMapDirectory working with a big
> segment, and after some failed attempts playing with maxChunkSize, I decided
> to switch to another FSDirectory implementation but I needed to do that on my
> own main.
> Should we also consider to use a FileSwitchDirectory?
> I'm willing to contribute, could you please let me know your thoughts about
> it?
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
