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

Dag H. Wanvik edited comment on DERBY-4741 at 11/15/10 2:08 PM:
----------------------------------------------------------------

Uploading patch b-03 which adds logic to RAFContainer4 to handle
properly the fact that a container can't be reopened:

    // In case the recovering thread can't successfully recover the container,
    // it will throw, so other waiting threads need to give up as well.  This
    // can happen at shutdown time when interrupts are used to stop threads.
    private boolean giveUpIO = false;
    private final Object giveUpIOm = new Object(); // its monitor

We make sure to notify any waiting threads when the recovering thread gives up 
and throws FILE_IO_INTERRUPTED. The waiting threads will next see giveUpIO == 
true, and give up as well.

This should make sure that threads not doing the recovery, will detect that 
recovery will not happen, and throw as well. SuitesAll finished successfully.


      was (Author: dagw):
    Uploading patch b-03 which adds logic to RAFContainer4 to handle
properly the fact that a container can't be reopened:

    // In case the recovering thread can't successfully recover the container,
    // it will throw, so other waiting threads need to give up as well.  This
    // can happen at shutdown time when interrupts are used to stop threads.
    private boolean giveUpIO = false;
    private final Object giveUpIOm = new Object(); // its monitor

We make sure to notify any waiting threads when the recovering thread throw. 
The waiting threads will next see giveUpIO == true, and give up as well.

This should make sure that threads not doing the recovery, will detect that 
recovery will not happen, and throw as well. SuitesAll finished successfully.

  
> 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-b-03-nio.diff, derby-4741-b-03-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