[ 
https://issues.apache.org/jira/browse/DERBY-4741?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12931526#action_12931526
 ] 

Dag H. Wanvik edited comment on DERBY-4741 at 11/12/10 4:22 PM:
----------------------------------------------------------------

Found that Derby uses interrupt to stop threads at shutdown[1]. The present 
logic in RAFContainer4 doesn't
take this into account, with the consequence that an application thread got 
stuck waiting for
a container recovery that never took place. We get an exception during 
execution of 
                       closeContainer();
                        openContainer(currentIdentity);
If this exception is not due to an interrupt, we throw FILE_IO_INTERRUPTED. 
Unfortunately, in this case,
the present patch neglects to release any threads waiting for recovery to 
finish, so they are stuck waiting for "restoreChannelInProgress".

Saw this during suitesAll.

[1] BaseMonitor#shutdown -> notifyAllActiveThreads and in 
DatabaseContextImpl#cleanupOnError->notifyAllActiveThreads


      was (Author: dagw):
    Found that Derby uses interrupt to stop threads at shutdown. The present 
logic in RAFContainer4 doesn't
take this into account, with the consequence that an application thread got 
stuck waiting for
a container recovery that never took place. We get an exception during 
execution of 
                       closeContainer();
                        openContainer(currentIdentity);
If this exception is not due to an interrupt, we throw FILE_IO_INTERRUPTED. 
Unfortunately, in this case,
the present patch neglects to release any threads waiting for recovery to 
finish, so they are stuck waiting for "restoreChannelInProgress".

Saw this during suitesAll.

  
> Make Derby work reliably in the presence of thread interrupts
> -------------------------------------------------------------
>
>                 Key: DERBY-4741
>                 URL: https://issues.apache.org/jira/browse/DERBY-4741
>             Project: Derby
>          Issue Type: Bug
>          Components: Store
>    Affects Versions: 10.2.1.6, 10.2.2.0, 10.3.1.4, 10.3.2.1, 10.3.3.0, 
> 10.4.1.3, 10.4.2.0, 10.5.1.1, 10.5.2.0, 10.5.3.0, 10.6.1.0
>            Reporter: Dag H. Wanvik
>            Assignee: Dag H. Wanvik
>         Attachments: derby-4741-a-01-api-interruptstatus.diff, 
> derby-4741-a-01-api-interruptstatus.stat, 
> derby-4741-a-02-api-interruptstatus.diff, 
> derby-4741-a-02-api-interruptstatus.stat, 
> derby-4741-a-03-api-interruptstatus.diff, 
> derby-4741-a-03-api-interruptstatus.stat, 
> derby-4741-a-04-api-interruptstatus.diff, 
> derby-4741-a-04-api-interruptstatus.stat, 
> derby-4741-all+lenient+resurrect.diff, derby-4741-all+lenient+resurrect.stat, 
> derby-4741-b-01-nio.diff, derby-4741-b-01-nio.stat, derby-4741-b-02-nio.diff, 
> derby-4741-b-02-nio.stat, 
> derby-4741-nio-container+log+waits+locks+throws.diff, 
> derby-4741-nio-container+log+waits+locks+throws.stat, 
> derby-4741-nio-container+log+waits+locks-2.diff, 
> derby-4741-nio-container+log+waits+locks-2.stat, 
> derby-4741-nio-container+log+waits+locks.diff, 
> derby-4741-nio-container+log+waits+locks.stat, 
> derby-4741-nio-container+log+waits.diff, 
> derby-4741-nio-container+log+waits.stat, derby-4741-nio-container+log.diff, 
> derby-4741-nio-container+log.stat, derby-4741-nio-container-2.diff, 
> derby-4741-nio-container-2.log, derby-4741-nio-container-2.stat, 
> derby-4741-nio-container-2b.diff, derby-4741-nio-container-2b.stat, 
> derby.log, derby.log, InterruptResilienceTest.java, MicroAPITest.java, 
> xsbt0.log.gz
>
>
> When not executing on a small device VM, Derby has been using the Java NIO 
> classes java.nio.clannel.* for file io.
> If thread is interrupted while executing blocking IO operations in NIO, the 
> ClosedByInterruptException will get thrown. Unfortunately, Derby isn't 
> current architected to retry and complete such operations (before passing on 
> the interrupt), so the Derby database can be left in an inconsistent state 
> and we therefore have to return a database level error. This means the 
> applications can no longer access the database without a shutdown and reboot 
> including a recovery.
> It would be nice if Derby could somehow detect and finish IO operations 
> underway when thread interrupts happen before passing the exception on to the 
> application. Derby embedded is sometimes embedded in applications that use 
> Thread.interrupt to stop threads.

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

Reply via email to