Robert Muir created LUCENE-4649:
-----------------------------------

             Summary: kill ThreadInterruptedException
                 Key: LUCENE-4649
                 URL: https://issues.apache.org/jira/browse/LUCENE-4649
             Project: Lucene - Core
          Issue Type: Bug
            Reporter: Robert Muir


the way we currently do this is bogus.

For example in FSDirectory's fsync, i think we should instead:

{noformat}
} catch (InterruptedException ie) {
- throw new ThreadInterruptedException(ie);
+ Thread.currentThread().interrupt(); // restore status
+ IOException e = new java.io.InterruptedIOException("fsync() interrupted");
+ e.initCause(ie);
+ throw e;
{noformat}

and crazy code in IndexWriter etc that catches ThreadInterruptedExc just to 
restore status should be removed. 

Instead the guy doing the catch (InterruptedException) should do the right 
thing.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

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

Reply via email to