Uploaded image for project: 'Lucene - Core'
  1. Lucene - Core
  2. LUCENE-5543

Remove Directory.fileExists

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 4.8, 6.0
    • core/store
    • None
    • New

    Description

      Since 3.0.x/3.6.x (see LUCENE-5541), Lucene has substantially removed
      its reliance on fileExists to the point where I think we can fully
      remove it now.

      Like the other iffy IO methods we've removed over time (touchFile,
      fileModified, seeking back during write, ...), File.exists is
      dangerous because a low level IO issue can cause it to return false
      when it should have returned true. The fewer IO operations we rely on
      the more reliable/portable Lucene is.

      Attachments

        1. LUCENE-5543.patch
          55 kB
          Michael McCandless

        Activity

          People

            mikemccand Michael McCandless
            mikemccand Michael McCandless
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: