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

Michael McCandless commented on LUCENE-793:
-------------------------------------------


You're right, technically this is an API change.

The only cases I changed were the ones stemming from LUCENE-140: the
original "docs out of order" corruption plus 2 other consisteny checks
I had added as part of the fix for LUCENE-140.

(There are other places where Lucene throws a IllegalStateException
that I did not change.)

I think it's extremely unlikely users are relying on the
IllegalStateException specifically (ie catching it explicitly and
doing something about it)?

EG these particular cases were never listed in the "throws"
(IllegalStateException is unchecked).  And if you hit this exception
your index truly is corrupt.

Since these really are cases of severe index corruption I thought it
best to throw CorruptIndexException instead?


> Javadocs should explain possible causes for IOExceptions
> --------------------------------------------------------
>
>                 Key: LUCENE-793
>                 URL: https://issues.apache.org/jira/browse/LUCENE-793
>             Project: Lucene - Java
>          Issue Type: Bug
>          Components: Javadocs
>            Reporter: Michael McCandless
>         Assigned To: Michael McCandless
>            Priority: Minor
>         Attachments: LUCENE-793.patch
>
>
> Most methods in Lucene reserve the right to throw an IOException.  This can 
> occur for nearly all methods from low level problems like wrong permissions, 
> transient IO errors, bad hard drive or corrupted file system, corrupted 
> index, etc, but for some methods there are also more interesting causes that 
> we should try to document.
> Spinoff of this thread:
>     http://www.gossamer-threads.com/lists/lucene/java-user/44929

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to