geronimo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From j...@codehaus.org
Subject [jira] Updated: (GERONIMO-95) [PATCH] DeploymentPlan may not query all tasks in canRun()
Date Sun, 28 Sep 2003 14:56:08 GMT
The following issue has been updated:

    Updater: Gareth Bryan (mailto:garethbryan@fastmail.fm)
       Date: Sun, 28 Sep 2003 9:54 AM
    Changes:
             description changed from In o.a.g.kernel.deployment.DeploymentPlan.canRun(),
incorrect usage of the short-circuit AND operator means that if a previous task reported that
it could not run, subsequent tasks will not be queried.

This is the opposite to the indened behaviour which is documented in the code comment "always
check each task, so the task can throw an exception if the task can never run".

The patch file is a trivial change from:

canRun = canRun && task.canRun()

to:

canRun = task.canRun() && canRun

The patch was generated by running "cvs diff -uN" from the project root directory (incubator-geronimo).

Also included in the patch and attached zip file is the testcase which brought this problem
to light.
    ---------------------------------------------------------------------
For a full history of the issue, see:

  http://jira.codehaus.org/secure/ViewIssue.jspa?key=GERONIMO-95&page=history

---------------------------------------------------------------------
View the issue:

  http://jira.codehaus.org/secure/ViewIssue.jspa?key=GERONIMO-95


Here is an overview of the issue:
---------------------------------------------------------------------
        Key: GERONIMO-95
    Summary: [PATCH] DeploymentPlan may not query all tasks in canRun()
       Type: Bug

     Status: Unassigned
   Priority: Minor

 Time Spent: Unknown
  Remaining: Unknown

    Project: Apache Geronimo
 Components: 
             core

   Assignee: 
   Reporter: Gareth Bryan

    Created: Sun, 28 Sep 2003 9:51 AM
    Updated: Sun, 28 Sep 2003 9:54 AM

Description:
In o.a.g.kernel.deployment.DeploymentPlan.canRun(), incorrect usage of the short-circuit AND
operator means that if a previous task reported that it could not run, subsequent tasks will
not be queried.

This is the opposite to the indened behaviour which is documented in the code comment "always
check each task, so the task can throw an exception if the task can never run".

The patch file is a trivial change from:

canRun = canRun && task.canRun()

to:

canRun = task.canRun() && canRun

The patch was generated by running "cvs diff -uN" from the project root directory (incubator-geronimo).

Also included in the patch and attached zip file is the testcase which brought this problem
to light.

Similarly, this zip file was created from the project root (incubator-geronimo).

Note: module affected is the kernel. Assigned to component 'core' due to lack of 'kernel'
component. Please shuffle as necessary.


---------------------------------------------------------------------
JIRA INFORMATION:
This message is automatically generated by JIRA.

If you think it was sent incorrectly contact one of the administrators:
   http://jira.codehaus.org/secure/Administrators.jspa

If you want more information on JIRA, or have a bug to report see:
   http://www.atlassian.com/software/jira


Mime
View raw message