[
https://issues.apache.org/jira/browse/OODT-414?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13231234#comment-13231234
]
[email protected] commented on OODT-414:
----------------------------------------------------
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4375/#review6036
-----------------------------------------------------------
Ship it!
Minor comments, LGTM.
- Chris
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
