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

Future instance returned for the async invocation does not honour javadoc about always returning true after the cancel() is invoked

    Details

    • Affects:
      Release Notes
    • Similar Issues:
      Show 8 results 

      Description

      After method cancel() returns, subsequent calls to isDone() should always return true.

      The server side implementation of the Future instance returned for the async invocation has a bug in the isDone() method since it doesn't honour the requirements set by the isDone() javadoc about always returning true after the cancel() is invoked.

      Please, see the comment https://issues.jboss.org/browse/EJBCLIENT-28?focusedCommentId=12690373&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-12690373
      On jira EJBCLIENT-28.

      The testcase could be checked here:
      https://github.com/ochaloup/jboss-as/blob/async-method-cancel-invocation/testsuite/integration/basic/src/test/java/org/jboss/as/test/integration/ejb/async/AsyncMethodTestCase.java#L184

        Issue Links

          Activity

          Hide
          jaikiran pai
          added a comment -
          Show
          jaikiran pai
          added a comment - Pull request sent https://github.com/jbossas/jboss-as/pull/2246
          Hide
          Shelly McGowan
          added a comment -

          Marking for CR1 and will verify it was brought in.

          Show
          Shelly McGowan
          added a comment - Marking for CR1 and will verify it was brought in.
          Hide
          jaikiran pai
          added a comment -

          This made it to AS7 upstream this morning. Marking this as resolved.

          Show
          jaikiran pai
          added a comment - This made it to AS7 upstream this morning. Marking this as resolved.
          Hide
          Ondřej Chaloupka
          added a comment -

          Verified on ER8.

          Show
          Ondřej Chaloupka
          added a comment - Verified on ER8.
          Hide
          Misty Stanley-Jones
          added a comment -

          Reopening for release note

          Show
          Misty Stanley-Jones
          added a comment - Reopening for release note
          Hide
          Ondřej Chaloupka
          added a comment -

          Verified on CR1 - org.jboss.as.test.integration.ejb.async.AsyncMethodTestCase.testCancelAsyncMethod() passed

          Show
          Ondřej Chaloupka
          added a comment - Verified on CR1 - org.jboss.as.test.integration.ejb.async.AsyncMethodTestCase.testCancelAsyncMethod() passed
          Hide
          RH Bugzilla Integration
          added a comment -

          This issue has been migrated to Bugzilla bug 900364. 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 900364 . 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:
              jaikiran pai
              Reporter:
              Ondřej Chaloupka
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: