Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.0, 6.0
    • Component/s: None
    • Labels:
      None
    • Lucene Fields:
      New

      Description

      Spinoff of LUCENE-4371:

      I don't think the default impl (SlicedIndexInput) should overrided BII's copyBytes? Seems ... spooky.

      There are copyBytes everywhere, mostly not really being used.
      Particularly DataOutput.copyBytes(DataInput) versus IndexInput.copyBytes(IndexOutput).

      Bulk merging already uses DataOutput.copyBytes(DataInput), its the most general (as it works on DataInput/Output), and its in dst, src order.

      I think we should remove IndexInput.copyBytes, its not necessary.

        Activity

        Hide
        Robert Muir added a comment -

        I want to fix this for 4.0 so we don't have to maintain deprecated implementations.

        Show
        Robert Muir added a comment - I want to fix this for 4.0 so we don't have to maintain deprecated implementations.
        Hide
        Michael McCandless added a comment -

        +1

        Show
        Michael McCandless added a comment - +1
        Hide
        Uwe Schindler added a comment -

        +1, this annoyed me since long time!

        Show
        Uwe Schindler added a comment - +1, this annoyed me since long time!
        Hide
        Commit Tag Bot added a comment -

        [branch_4x commit] Robert Muir
        http://svn.apache.org/viewvc?view=revision&revision=1383911

        LUCENE-4377: Remove IndexInput.copyBytes(IndexOutput, long)

        Show
        Commit Tag Bot added a comment - [branch_4x commit] Robert Muir http://svn.apache.org/viewvc?view=revision&revision=1383911 LUCENE-4377 : Remove IndexInput.copyBytes(IndexOutput, long)
        Hide
        Uwe Schindler added a comment -

        Closed after release.

        Show
        Uwe Schindler added a comment - Closed after release.

          People

          • Assignee:
            Robert Muir
            Reporter:
            Robert Muir
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development