Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 4.0-ALPHA, master
    • Fix Version/s: 4.0-ALPHA, master
    • Component/s: core/index
    • Labels:
      None
    • Lucene Fields:
      New

      Description

      The Payload class offers a very similar functionality to BytesRef. The code internally uses BytesRef-s to represent payloads, and on indexing and on retrieval this data is repackaged from/to Payload.

      This seems wasteful. I propose to remove the Payload class and use BytesRef instead, thus avoid this re-wrapping and reducing the API footprint.

      1. LUCENE-4122.patch
        69 kB
        Andrzej Bialecki

        Activity

        Andrzej Bialecki created issue -
        Hide
        Andrzej Bialecki added a comment -

        Patch for trunk. All tests pass.

        Show
        Andrzej Bialecki added a comment - Patch for trunk. All tests pass.
        Andrzej Bialecki made changes -
        Field Original Value New Value
        Attachment LUCENE-4122.patch [ 12531411 ]
        Hide
        Robert Muir added a comment -

        +1

        Show
        Robert Muir added a comment - +1
        Hide
        Michael McCandless added a comment -

        +1

        Show
        Michael McCandless added a comment - +1
        Hide
        Andrzej Bialecki added a comment -

        Committed in rev. 1348171 to trunk and in rev. 1348227 to branch_4x.

        Show
        Andrzej Bialecki added a comment - Committed in rev. 1348171 to trunk and in rev. 1348227 to branch_4x.
        Andrzej Bialecki made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Uwe Schindler made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        3h 46m 1 Andrzej Bialecki 08/Jun/12 20:19
        Resolved Resolved Closed Closed
        335d 14h 21m 1 Uwe Schindler 10/May/13 10:40

          People

          • Assignee:
            Andrzej Bialecki
            Reporter:
            Andrzej Bialecki
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development