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

[email protected] commented on OODT-414:
----------------------------------------------------



bq.  On 2012-03-16 14:28:24, Chris Mattmann wrote:
bq.  > trunk/pge/src/test/org/apache/oodt/cas/pge/TestPGETaskInstance.java, 
line 108
bq.  > <https://reviews.apache.org/r/4375/diff/2/?file=93053#file93053line108>
bq.  >
bq.  >     what makes this throw an exception now?

I'm gradually adding "throws Exception" to all the methods in CAS-PGE's 
PGETaskInstance so if they are overridden by the extending class, the extending 
class can easily kill CAS-PGE (and give the reason why it should stop) by just 
throwing any Exception


- brian


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


On 2012-03-16 08:25:17, brian Foster wrote:
bq.  
bq.  -----------------------------------------------------------
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/4375/
bq.  -----------------------------------------------------------
bq.  
bq.  (Updated 2012-03-16 08:25:17)
bq.  
bq.  
bq.  Review request for oodt, Chris Mattmann, Ricky Nguyen, Paul Ramirez, and 
Thomas Bennett.
bq.  
bq.  
bq.  Summary
bq.  -------
bq.  
bq.  This handler will allow for multiple CAS-PGE instances to exist in the 
same JVM while routing the logging from each to their own log files in their 
execution directory
bq.  
bq.  -----
bq.  This patch also includes:
bq.   - adding support for PgeMetadata to take PgeTaskMetKeys directly
bq.   - some cleanup and reorganizing of PgeTaskInstance
bq.  
bq.  
bq.  This addresses bug OODT-414.
bq.      https://issues.apache.org/jira/browse/OODT-414
bq.  
bq.  
bq.  Diffs
bq.  -----
bq.  
bq.    trunk/pge/src/main/java/org/apache/oodt/cas/pge/PGETaskInstance.java 
1301267 
bq.    
trunk/pge/src/main/java/org/apache/oodt/cas/pge/logging/PgeLogHandler.java 
PRE-CREATION 
bq.    
trunk/pge/src/main/java/org/apache/oodt/cas/pge/logging/PgeLogRecord.java 
PRE-CREATION 
bq.    
trunk/pge/src/main/java/org/apache/oodt/cas/pge/metadata/PgeMetadata.java 
1300798 
bq.    
trunk/pge/src/main/java/org/apache/oodt/cas/pge/metadata/PgeTaskMetKeys.java 
1301267 
bq.    trunk/pge/src/test/org/apache/oodt/cas/pge/TestPGETaskInstance.java 
1301267 
bq.    
trunk/pge/src/test/org/apache/oodt/cas/pge/logging/TestPgeLogHandler.java 
PRE-CREATION 
bq.  
bq.  Diff: https://reviews.apache.org/r/4375/diff
bq.  
bq.  
bq.  Testing
bq.  -------
bq.  
bq.  Added and updated unit-tests
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  brian
bq.  
bq.


                
> Create Java Logger Handler for CAS-PGE
> --------------------------------------
>
>                 Key: OODT-414
>                 URL: https://issues.apache.org/jira/browse/OODT-414
>             Project: OODT
>          Issue Type: Sub-task
>          Components: pge wrapper framework
>    Affects Versions: 0.3
>         Environment: none
>            Reporter: Brian Foster
>            Assignee: Brian Foster
>            Priority: Minor
>             Fix For: 0.4
>
>         Attachments: OODT-414.2012-03-15.txt
>
>
> This handler will allow for multiple CAS-PGE instances to exist in the same 
> JVM while routing the logging from each to their own log files in their 
> execution directory

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

        

Reply via email to