lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Shai Erera (JIRA)" <>
Subject [jira] [Commented] (LUCENE-4304) Remove PayloadProcessProvider
Date Mon, 13 Aug 2012 18:31:38 GMT


Shai Erera commented on LUCENE-4304:

The facet module comes with an example code for merging a taxonomy index (followed by IW.addIndexes(Dir)).
It'd be good if in the process of removing PPP we can still keep the example, only implemented
with a FilterAtomicReader. I think FAR is better than Codec because it doesn't require you
to open an IW with that Codec set, rather you can just IW.addIndexes(IndexReader).
> Remove PayloadProcessProvider
> -----------------------------
>                 Key: LUCENE-4304
>                 URL:
>             Project: Lucene - Core
>          Issue Type: Bug
>            Reporter: Michael McCandless
> Now that we have pluggable codecs (well, PostingsFormat), an app should use a custom
PostingsFormat if it really must change payloads while merging.
> Alternatively, use a FilteredIndexReader to modify anything during addIndexes (eg the
facets use case, modifying payloads).
> Since this capability can be handled by existing more-generic functions I don't see why
we need to keep PPP around in core.  PPP is also fragile because an app generally has no visibility
on when a merge commits so it can't know if the payloads it retrieves are pre or post PPP.
> I think merging shouldn't change postings as a side-effect (by default, anyway, since
a custom PF can of course override merge and do whatever it wants).

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:!default.jspa
For more information on JIRA, see:


To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message