Uploaded image for project: '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

      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

        Gliffy Diagrams

          Issue Links

            Activity

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

            Marking for CR1 and will verify it was brought in.

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

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

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

            Verified on ER8.

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

            Reopening for release note

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

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

            Show
            ochaloup Ondřej Chaloupka added a comment - Verified on CR1 - org.jboss.as.test.integration.ejb.async.AsyncMethodTestCase.testCancelAsyncMethod() passed
            Hide
            jira-bugzilla-migration 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
            jira-bugzilla-migration 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 jaikiran pai
                Reporter:
                ochaloup Ondřej Chaloupka
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Development