[dna-issues] [JBoss JIRA] Commented: (DNA-217) Connector SPI should have protocol for cancelling operations

Randall Hauch (JIRA) jira-events at lists.jboss.org
Thu Sep 4 10:35:42 EDT 2008


    [ https://jira.jboss.org/jira/browse/DNA-217?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12428259#action_12428259 ] 

Randall Hauch commented on DNA-217:
-----------------------------------

Removed the use of the InterruptedException from most of the connector SPI methods.  (It is still used on "ping(), since that's more akin to other interruptable methods).

> Connector SPI should have protocol for cancelling operations
> ------------------------------------------------------------
>
>                 Key: DNA-217
>                 URL: https://jira.jboss.org/jira/browse/DNA-217
>             Project: DNA
>          Issue Type: Task
>          Components: Connectors, SPI
>    Affects Versions: 0.2
>            Reporter: Randall Hauch
>            Assignee: Randall Hauch
>             Fix For: 0.2
>
>
> The Connector SPI needs a specific way for calls to the connector to be canceled.  Currently, most of the methods throw InterruptedException, which is a painful (and confusing) way to handle it.  This exception should be removed while the use of ProgressMonitor be added.  
> Additionally, we may want to consider modifying the ProgressMonitor framework so that it's possible for it to threat Thread.interrupt() as a cancellation (whereby it would use that, in addition to the internal flag, to determine whether the monitor is canceled.  This may require an additional method whereby the code checking "isCanceled()" acknowledges that it is canceling what it's doing (so that the ProgressMonitor can call Thread.interrupted() to clear the interrupted state).

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        



More information about the dna-issues mailing list