geronimo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
Subject [jira] Closed: (GERONIMO-95) [PATCH] DeploymentPlan may not query all tasks in canRun()
Date Sun, 07 Dec 2003 03:41:56 GMT

   The following issue has been closed.

   Resolver: Alan Cabrera
       Date: Sat, 6 Dec 2003 9:41 PM

Thanks Gareth Bryan!
View the issue:

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

     Status: Closed
   Priority: Minor
 Resolution: FIXED

 Original Estimate: Unknown
 Time Spent: Unknown
  Remaining: Unknown

    Project: Apache Geronimo

   Assignee: Alan Cabrera
   Reporter: Gareth Bryan

    Created: Sun, 28 Sep 2003 9:51 AM
    Updated: Sat, 6 Dec 2003 9:41 PM

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()


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.

This message is automatically generated by JIRA.

If you think it was sent incorrectly contact one of the administrators:

If you want more information on JIRA, or have a bug to report see:

View raw message