lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Shai Erera (Updated) (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (LUCENE-3620) FilterIndexReader does not override all of IndexReader methods
Date Wed, 07 Dec 2011 14:46:39 GMT

     [ https://issues.apache.org/jira/browse/LUCENE-3620?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Shai Erera updated LUCENE-3620:
-------------------------------

    Attachment: LUCENE-3620-trunk.patch

Patch adds the test to TestFilterIndexReader. Uwe asked that I do not commit these changes
(test + FIR/IR fixes) until he merges in the branch on IR-read-only. We decided that Uwe will
apply that patch to the branch, fix FIR/IR there and merge the branch afterwards.
                
> FilterIndexReader does not override all of IndexReader methods
> --------------------------------------------------------------
>
>                 Key: LUCENE-3620
>                 URL: https://issues.apache.org/jira/browse/LUCENE-3620
>             Project: Lucene - Java
>          Issue Type: Bug
>          Components: core/search
>            Reporter: Shai Erera
>            Assignee: Shai Erera
>            Priority: Minor
>             Fix For: 3.6, 4.0
>
>         Attachments: LUCENE-3620-trunk.patch, LUCENE-3620.patch, LUCENE-3620.patch, LUCENE-3620.patch
>
>
> FilterIndexReader does not override all of IndexReader methods. We've hit an error in
LUCENE-3573 (and fixed it). So I thought to write a simple test which asserts that FIR overrides
all methods of IR (and we can filter our methods that we don't think that it should override).
The test is very simple (attached), and it currently fails over these methods:
> {code}
> getRefCount
> incRef
> tryIncRef
> decRef
> reopen
> reopen
> reopen
> reopen
> clone
> numDeletedDocs
> document
> setNorm
> setNorm
> termPositions
> deleteDocument
> deleteDocuments
> undeleteAll
> getIndexCommit
> getUniqueTermCount
> getTermInfosIndexDivisor
> {code}
> I didn't yet fix anything in FIR -- if you spot a method that you think we should not
override and delegate, please comment.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org


Mime
View raw message