[
https://issues.apache.org/jira/browse/LUCENE-6124?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Robert Muir updated LUCENE-6124:
--------------------------------
Attachment: LUCENE-6124.patch
Here is current patch, with those tests passing. BlockTree, NRTCachingDir,
MockDir, etc had bugs.
We need to add checks in e.g. BasePostingsFormatTestCase and all those kinds of
tests too, because I know there are other term dictionaries with the same
pattern blocktree has.
> Fix broken close() methods
> --------------------------
>
> Key: LUCENE-6124
> URL: https://issues.apache.org/jira/browse/LUCENE-6124
> Project: Lucene - Core
> Issue Type: Bug
> Reporter: Robert Muir
> Attachments: LUCENE-6124.patch, LUCENE-6124_asserts.patch,
> LUCENE-6124_asserts.patch
>
>
> Closeable.close() says "If the stream is already closed then invoking this
> method has no effect.".
> But a lot of our code does not really respect that. If i add an "extra"
> close() call in assertingcodec, it finds all kinds of bugs in codec code, for
> example:
> {noformat}
> [junit4] Tests with failures (first 10 out of 59):
> [junit4] -
> org.apache.lucene.index.TestCrashCausesCorruptIndex.testCrashCorruptsIndexing
> [junit4] -
> org.apache.lucene.codecs.asserting.TestAssertingPostingsFormat.testDocsOnly
> [junit4] -
> org.apache.lucene.codecs.asserting.TestAssertingPostingsFormat.testDocsAndFreqsAndPositionsAndOffsetsAndPayloads
> [junit4] -
> org.apache.lucene.codecs.asserting.TestAssertingPostingsFormat.testDocsAndFreqs
> [junit4] -
> org.apache.lucene.codecs.asserting.TestAssertingPostingsFormat.testDocsAndFreqsAndPositionsAndOffsets
> [junit4] -
> org.apache.lucene.codecs.asserting.TestAssertingPostingsFormat.testRandom
> [junit4] -
> org.apache.lucene.codecs.asserting.TestAssertingPostingsFormat.testDocsAndFreqsAndPositionsAndPayloads
> [junit4] -
> org.apache.lucene.codecs.asserting.TestAssertingPostingsFormat.testDocsAndFreqsAndPositions
> [junit4] - org.apache.lucene.index.TestDirectoryReader.testFilesOpenClose
> [junit4] -
> org.apache.lucene.index.TestIndexWriterDelete.testIndexingThenDeleting
> {noformat}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
