lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Adrien Grand (JIRA)" <>
Subject [jira] [Commented] (LUCENE-7283) Move SlowCompositeReaderWrapper to solr sources
Date Sat, 14 May 2016 13:16:12 GMT


Adrien Grand commented on LUCENE-7283:

I think it is fine to remove it from Lucene even if it is used in the wild. After all, our
goal is that users stop using this class. If someone really needs the functionality, they
can still fork the code? 

> Move SlowCompositeReaderWrapper to solr sources
> -----------------------------------------------
>                 Key: LUCENE-7283
>                 URL:
>             Project: Lucene - Core
>          Issue Type: Improvement
>            Reporter: Michael McCandless
> Spinoff from LUCENE-6766, where we fixed index-time sorting to have first class support
in Lucene's ore, and no longer use {{SlowCompositeReaderWrapper}}.
> This is a dangerous, long living class, that tries to pretend a set of N segments is
actually just a single segment.  It's a leaky abstraction, has poor performance, and puts
undue pressure on the APIs of new Lucene features to try to keep up this illusion.
> With LUCENE-6766, finally all usage of this class (except for {{UninvertedReader}} tests,
which should maybe also move out?) has been removed from Lucene, so I think we should move
it to Solr.  This may also lead to a solution for LUCENE-7086 since e.g. the class could tap
into solr's schema to "know" how to handle points fields properly.

This message was sent by Atlassian JIRA

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

View raw message