commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Morten Hattesen (JIRA)" <>
Subject [jira] [Created] (IO-357) Tailer: Interruption while sleeping is silently ignored
Date Tue, 20 Nov 2012 23:46:58 GMT
Morten Hattesen created IO-357:

             Summary: Tailer: Interruption while sleeping is silently ignored
                 Key: IO-357
             Project: Commons IO
          Issue Type: Bug
          Components: Streams/Writers
    Affects Versions: 2.4
            Reporter: Morten Hattesen

The implementation of silently ignores interruptions while
sleeping (in two places).

Source snippet:
360	                    try {
361	                        Thread.sleep(delayMillis);
362	                    } catch (InterruptedException e) {
363	                    }
425	                try {
426	                    Thread.sleep(delayMillis);
427	                } catch (InterruptedException e) {
428	                }

This is an inappropriate behavior, since it prevents controlled shutdown by a container.

This may be rectified in one of these ways:

# Declare the method as "throws InterruptedException" and eliminate the catch clause. This
will ensure that a thread interruption (possibly caused by the forced shutdown by a container)
will cause processing to stop, and shutdown to proceed. Problem: Requires backwards incompatible
change to method signature.
# Rethrow the InterruptedException wrapped in a (subclass of) RuntimeException.
# Reassert the interrupted state of the thread by calling {{Thread.currentThread.interrupt()}}

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:

View raw message