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

Investigate to use FileoutputStream instead of RandomAccessFile in FSIndexOutput

    XMLWordPrintableJSON

Details

    • Task
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • 4.9, 6.0
    • core/store
    • None
    • New

    Description

      We no longer allow seeking in IndexOutput, so there is no need to use RandomAccessFile. We can change this with a < 1 KiB patch.

      Further improvements would be to merge this with OutputStreamIndexOutput, so we get many simplifications.

      There is also no reason anymore to separate DataOutput from IndexOutput. The only additional thing is IndexOutput#getFilePointer(), which is handled by an internal counter (does not use getFilePointer of the underlying RAF) and checksums.

      Attachments

        1. LUCENE-5678.patch
          2 kB
          Uwe Schindler
        2. LUCENE-5678.patch
          14 kB
          Uwe Schindler
        3. LUCENE-5678.patch
          14 kB
          Uwe Schindler
        4. LUCENE-5678.patch
          23 kB
          Uwe Schindler
        5. LUCENE-5678.patch
          23 kB
          Uwe Schindler
        6. LUCENE-5678.patch
          23 kB
          Uwe Schindler
        7. LUCENE-5678-4x.patch
          34 kB
          Uwe Schindler

        Issue Links

          contains

          Improvement - An improvement or enhancement to an existing feature or task. LUCENE-5582 Remove IndexOutput.length and .setLength

          • Major - Major loss of function.
          • Closed
          depends upon

          Bug - A problem which impairs or prevents the functions of the product. LUCENE-5727 Remove IndexOutput.seek

          • Major - Major loss of function.
          • Resolved
          relates to

          Bug - A problem which impairs or prevents the functions of the product. LUCENE-5164 Remove the OOM catching in SimpleFSDirectory and NIOFSDirectory

          • Major - Major loss of function.
          • Closed

          Activity

            People

              uschindler Uwe Schindler
              uschindler Uwe Schindler
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: