[
https://issues.apache.org/jira/browse/LUCENE-761?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12465753
]
Michael Busch commented on LUCENE-761:
--------------------------------------
Grant,
your are absolutely right, 755 does not block this issue. The reason why I
wanted to wait to submit a patch here was that 755 and this one are changing
the same files. So committing this one would have prevented 755 from applying
cleanly on the trunk. But since there were a couple of commits in the last
days/weeks and the Payloads API is still under discussion I can as well submit
a patch here now, because I have to change 755 to apply cleanly to the trunk
anyway.
> Clone proxStream lazily in SegmentTermPositions
> -----------------------------------------------
>
> Key: LUCENE-761
> URL: https://issues.apache.org/jira/browse/LUCENE-761
> Project: Lucene - Java
> Issue Type: Improvement
> Components: Index
> Reporter: Michael Busch
> Assigned To: Michael Busch
> Priority: Minor
>
> In SegmentTermPositions the proxStream should be cloned lazily, i. e. at the
> first time nextPosition() is called. Then the initialization costs of
> TermPositions are not higher anymore compared to TermDocs and thus there is
> no reason anymore for Scorers to use TermDocs instead of TermPositions. In
> fact, all Scorers should use TermPositions, because custom subclasses of
> existing scorers might want to access payloads, which is only possible via
> TermPositions. We could further merge SegmentTermDocs and
> SegmentTermPositions into one class and deprecate the interface TermDocs.
> I'm going to attach a patch once the payloads feature (LUCENE-755) is
> committed.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
https://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
