oodt-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "jiraposter@reviews.apache.org (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (OODT-310) Port WEngine to trunk
Date Wed, 02 May 2012 19:20:51 GMT

    [ https://issues.apache.org/jira/browse/OODT-310?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13266844#comment-13266844
] 

jiraposter@reviews.apache.org commented on OODT-310:
----------------------------------------------------


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4961/#review7482
-----------------------------------------------------------


like the patch!... few comments


./trunk/workflow/src/main/java/org/apache/oodt/cas/workflow/engine/TaskQuerier.java
<https://reviews.apache.org/r/4961/#comment16558>

    Why do you want Holding category to be returned?



./trunk/workflow/src/main/java/org/apache/oodt/cas/workflow/engine/TaskQuerier.java
<https://reviews.apache.org/r/4961/#comment16556>

    This should return a copy of runnableProcessors here... probably where your synchronization
issue is



./trunk/workflow/src/test/org/apache/oodt/cas/workflow/engine/TestTaskQuerier.java
<https://reviews.apache.org/r/4961/#comment16554>

    getRunnableProcessors() is never null... probably should be a unit-test to insure that
functionality



./trunk/workflow/src/test/org/apache/oodt/cas/workflow/engine/TestTaskQuerier.java
<https://reviews.apache.org/r/4961/#comment16555>

    !querier.getRunnableProcessors().isEmpty()



./trunk/workflow/src/test/org/apache/oodt/cas/workflow/engine/TestTaskQuerier.java
<https://reviews.apache.org/r/4961/#comment16565>

    When Success/done is passed in, the processor created is incorrect since it has a sub-processor
in Queued/waiting



./trunk/workflow/src/test/org/apache/oodt/cas/workflow/engine/TestTaskQuerier.java
<https://reviews.apache.org/r/4961/#comment16560>

    Should probably add a NOT done processor here


- brian


On 2012-05-02 05:08:45, Chris Mattmann wrote:
bq.  
bq.  -----------------------------------------------------------
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/4961/
bq.  -----------------------------------------------------------
bq.  
bq.  (Updated 2012-05-02 05:08:45)
bq.  
bq.  
bq.  Review request for oodt, brian Foster, Ricky Nguyen, Paul Ramirez, Sheryl John, and Thomas
Bennett.
bq.  
bq.  
bq.  Summary
bq.  -------
bq.  
bq.  Task Querier thread for OODT-310. See javadocs on: https://builds.apache.org/job/oodt-trunk/javadoc/org/apache/oodt/cas/workflow/engine/TaskQuerier.html
bq.  
bq.  
bq.  This addresses bug OODT-310.
bq.      https://issues.apache.org/jira/browse/OODT-310
bq.  
bq.  
bq.  Diffs
bq.  -----
bq.  
bq.    ./trunk/workflow/src/main/java/org/apache/oodt/cas/workflow/engine/TaskQuerier.java
1332505 
bq.    ./trunk/workflow/src/main/java/org/apache/oodt/cas/workflow/engine/WorkflowProcessor.java
1331866 
bq.    ./trunk/workflow/src/test/org/apache/oodt/cas/workflow/engine/TestTaskQuerier.java
PRE-CREATION 
bq.  
bq.  Diff: https://reviews.apache.org/r/4961/diff
bq.  
bq.  
bq.  Testing
bq.  -------
bq.  
bq.  Includes unit test, that currently isn't passing. I think I know why (something up with
my threading logic and synchronized keywords) but wanted to throw it up for review. I'll likely
be working on this tomorrow or the following evening.
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  Chris
bq.  
bq.


                
> Port WEngine to trunk
> ---------------------
>
>                 Key: OODT-310
>                 URL: https://issues.apache.org/jira/browse/OODT-310
>             Project: OODT
>          Issue Type: Sub-task
>          Components: workflow manager
>            Reporter: Chris A. Mattmann
>            Assignee: Chris A. Mattmann
>             Fix For: 0.5
>
>         Attachments: OODT-310.Mattmann.082311.patch.txt, OODT-310.Mattmann.100911.patch.txt,
OODT-310.sherylj.101711.patch.txt
>
>
> This issue will track the porting of the wengine-branch WorkflowEngine interface and
associated classes to trunk. This involves changing to the Processor model (Sequential, Condition,
etc.) which has already begun (as of OODT-70), and also the actual Engine implementation itself,
that doesn't block and that includes a queue-based model developed by [~bfoster].

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Mime
View raw message