lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Michael McCandless (JIRA)" <>
Subject [jira] Updated: (LUCENE-1658) Absorb NIOFSDirectory into FSDirectory
Date Mon, 25 May 2009 20:00:45 GMT


Michael McCandless updated LUCENE-1658:

    Attachment: LUCENE-1658.patch

Attached patch:

  * I created SimpleFSDir (the renaming for FSDir), and left FSDir to
    be the base class (in 3.0 it will become abstract).  FSDir
    implements most methods, handles locking, etc., so the subclass
    will just have to implement openInput/createOutput.

  * Added static methods; currently they simply choose
    NIOFSDir on non-Windows OS, and FSDir on Windows OS.  I was
    tempted to choose MMapDir on 64 bit Windows JRE, but it makes me

  * Strengthened javadocs

  * Cutover most unit tests to

> Absorb NIOFSDirectory into FSDirectory
> --------------------------------------
>                 Key: LUCENE-1658
>                 URL:
>             Project: Lucene - Java
>          Issue Type: Improvement
>          Components: Store
>            Reporter: Michael McCandless
>            Assignee: Michael McCandless
>            Priority: Minor
>             Fix For: 2.9
>         Attachments: LUCENE-1658.patch, LUCENE-1658.patch
> I think whether one uses* vs java.nio.* or eventually
> java.nio2.*, or some other means, is an under-the-hood implementation
> detail of FSDirectory and doesn't merit a whole separate class.
> I think FSDirectory should be the core class one uses when one's index
> is in the filesystem.
> So, I'd like to deprecate NIOFSDirectory, absorbing it into
> FSDirectory, and add a setting "useNIO" to FSDirectory.  It should
> default to "true" for non-Windows OSs, because it gives far better
> concurrent performance on all platforms but Windows (due to known Sun
> JRE issue

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

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

View raw message