Details
-
Type:
Improvement
-
Status: Closed
-
Priority:
Major
-
Resolution: Fixed
-
Affects Version/s: 4.0-ALPHA, 6.0
-
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.
Activity
- All
- Comments
- Work Log
- History
- Activity
- Transitions
+1
Show
Robert Muir
added a comment - +1
+1
Show
Michael McCandless
added a comment - +1
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.
Patch for trunk. All tests pass.