[ 
https://issues.apache.org/jira/browse/LUCENE-8503?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16617332#comment-16617332
 ] 

Simon Willnauer commented on LUCENE-8503:
-----------------------------------------

I have to revert my statement. I was under the assumption that this happens in 
way more places than expected. Maybe I was way too deep in it to see it all 
over the place. :) Nevermind. I will change my pr to only fix the calls to 
reader.in. instead and keep the rest like it is.

> Simplify unwrapping Filter*Reader
> ---------------------------------
>
>                 Key: LUCENE-8503
>                 URL: https://issues.apache.org/jira/browse/LUCENE-8503
>             Project: Lucene - Core
>          Issue Type: Improvement
>    Affects Versions: 7.5, master (8.0)
>            Reporter: Simon Willnauer
>            Priority: Major
>             Fix For: 7.6, master (8.0)
>
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
>  Today we have 3 different kinds of FilterIndexReader. While 
> FilterDirecotryReader
>     and FilterLeafReader are simple to distinguish, FilterCodecReader make 
> decision harder
>     since now we need instanceof checks to deside which unwrap method we 
> should call. This
>     adds a simple interface that allows to build generic unwrap methods to 
> access the delegat
>     of each of the filtering readers.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to