JBoss Enterprise Application Platform 6
  1. JBoss Enterprise Application Platform 6
  2. JBPAPP6-746

isDone() method doesn't return true after cancel() call on EJB asynchronous method

    Details

    • Type: Bug Bug
    • Status: Closed Closed (View Workflow)
    • Priority: Major Major
    • Resolution: Done
    • Affects Version/s: EAP 6.0.0 DR 13
    • Fix Version/s: EAP 6.0.0 ER 8
    • Component/s: EJB
    • Security Level: Public (Everyone can see)
    • Labels:
      None
    • Similar Issues:
      Show 9 results 

      Description

      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."

        Issue Links

          Activity

          Hide
          Rajesh Rajasekaran
          added a comment -

          The upstream JIRA has been fixed in ejbclient 1.0.6. Assigning to Shelly and scheduling for ER7 to pull in the upstream release.

          Show
          Rajesh Rajasekaran
          added a comment - The upstream JIRA has been fixed in ejbclient 1.0.6. Assigning to Shelly and scheduling for ER7 to pull in the upstream release.
          Hide
          Shelly McGowan
          added a comment -

          EJB Client v 1.0.8 will be in ER7.

          Show
          Shelly McGowan
          added a comment - EJB Client v 1.0.8 will be in ER7.
          Hide
          Shelly McGowan
          added a comment -

          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.

          Show
          Shelly McGowan
          added a comment - 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.
          Hide
          Ondřej Chaloupka
          added a comment -

          Verified on ER8 - AsyncMethodTestCase.testCancelAsyncMethod() passes and seems OK.

          Show
          Ondřej Chaloupka
          added a comment - Verified on ER8 - AsyncMethodTestCase.testCancelAsyncMethod() passes and seems OK.
          Hide
          RH Bugzilla Integration
          added a comment -

          This issue has been migrated to Bugzilla bug 899843. Please note that this JIRA issue has been closed as part of the migration and therefore you will need to check the Bugzilla issue to find the current status.

          Show
          RH Bugzilla Integration
          added a comment - This issue has been migrated to Bugzilla bug 899843 . Please note that this JIRA issue has been closed as part of the migration and therefore you will need to check the Bugzilla issue to find the current status.

            People

            • Assignee:
              Shelly McGowan
              Reporter:
              Ondřej Chaloupka
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: