Uploaded image for project: '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
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: EAP 6.0.0 DR 13
    • Fix Version/s: EAP 6.0.0 ER 8
    • Component/s: EJB
    • Labels:
      None

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

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            rrajesh 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
            rrajesh 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 Shelly McGowan added a comment -

            EJB Client v 1.0.8 will be in ER7.

            Show
            shelly.mcgowan Shelly McGowan added a comment - EJB Client v 1.0.8 will be in ER7.
            Hide
            shelly.mcgowan 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 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
            ochaloup Ondřej Chaloupka added a comment -

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

            Show
            ochaloup Ondřej Chaloupka added a comment - Verified on ER8 - AsyncMethodTestCase.testCancelAsyncMethod() passes and seems OK.
            Hide
            jira-bugzilla-migration 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
            jira-bugzilla-migration 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 Shelly McGowan
                Reporter:
                ochaloup Ondřej Chaloupka
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Development