Bug 899843 (JBPAPP6-746) - isDone() method doesn't return true after cancel() call on EJB asynchronous method
Summary: isDone() method doesn't return true after cancel() call on EJB asynchronous m...
Keywords:
Status: CLOSED NEXTRELEASE
Alias: JBPAPP6-746
Product: JBoss Enterprise Application Platform 6
Classification: JBoss
Component: EJB
Version: 6.0.0
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: ---
: EAP 6.0.0
Assignee: Shelly McGowan
QA Contact:
URL: http://jira.jboss.org/jira/browse/JBP...
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-02-16 12:42 UTC by Ondrej Chaloupka
Modified: 2014-06-28 12:44 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-05-18 10:59:05 UTC
Type: Bug
Embargoed:

Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker JBPAPP6-746 0 None None None Never

Description Ondrej Chaloupka 2012-02-16 12:42:23 UTC 
project_key: JBPAPP6

Method isDone() does not return true after call of cancel method() on Future object bound to call of EJB asynchronous method.
Please check my test:
https://github.com/ochaloup/jboss-as/blob/JBQA-5271-tests-async-pull/testsuite/integration/basic/src/test/java/org/jboss/as/test/integration/ejb/async/AsyncMethodTestCase.java#L196

It's against documentation of cancel method (http://docs.oracle.com/javase/6/docs/api/java/util/concurrent/Future.html#cancel%28boolean%29) where is said:
"After this method returns, subsequent calls to isDone() will always return true. Subsequent calls to isCancelled() will always return true if this method returned true."
Comment 1 Ondrej Chaloupka 2012-02-16 12:42:39 UTC 
Link: Added: This issue Cloned from AS7-3750
Comment 2 Rajesh Rajasekaran 2012-05-01 17:17:14 UTC 
The upstream JIRA has been fixed in ejbclient 1.0.6. Assigning to Shelly and scheduling for ER7 to pull in the upstream release.
Comment 3 Shelly McGowan 2012-05-04 19:37:42 UTC 
EJB Client v 1.0.8 will be in ER7.
Comment 4 Shelly McGowan 2012-05-07 17:55:08 UTC 
Another pull request associated to this will be in CR1 as seen in JBPAPP-8920
https://github.com/jbossas/jboss-as/pull/2246

so moving the Fix in Version to CR for the retest.
Comment 5 Ondrej Chaloupka 2012-05-18 10:59:05 UTC 
Verified on ER8 - AsyncMethodTestCase.testCancelAsyncMethod() passes and seems OK.
Comment 6 Anne-Louise Tangring 2012-11-05 17:51:04 UTC 
Docs QE Status: Removed: NEW
Note You need to log in before you can comment on or make changes to this bug.