You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@oodt.apache.org by "Brian Foster (Created) (JIRA)" <ji...@apache.org> on 2012/03/18 21:58:40 UTC
[jira] [Created] (OODT-423) Port LoggerOutputStream in CAS-PGE to
existing LoggerOutputStream in oodt-commons io package
Port LoggerOutputStream in CAS-PGE to existing LoggerOutputStream in oodt-commons io package
--------------------------------------------------------------------------------------------
Key: OODT-423
URL: https://issues.apache.org/jira/browse/OODT-423
Project: OODT
Issue Type: Sub-task
Components: commons, pge wrapper framework
Affects Versions: 0.3
Environment: none
Reporter: Brian Foster
Assignee: Brian Foster
Priority: Minor
Fix For: 0.4
This will include porting unit-tests as well from CAS-PGE... CAS-PGE will be switched back to using oodt-commons LoggerOutputStream as well in this bug... CAS-PGE's LoggerOutputStream will be deleted.
--
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
[jira] [Commented] (OODT-423) Port LoggerOutputStream in CAS-PGE to
existing LoggerOutputStream in oodt-commons io package
Posted by "jiraposter@reviews.apache.org (Commented) (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OODT-423?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13232373#comment-13232373 ]
jiraposter@reviews.apache.org commented on OODT-423:
----------------------------------------------------
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4398/
-----------------------------------------------------------
Review request for oodt, Chris Mattmann, Ricky Nguyen, Paul Ramirez, and Thomas Bennett.
Summary
-------
This is the CAS-PGE portion of this change
This addresses bug OODT-423.
https://issues.apache.org/jira/browse/OODT-423
Diffs
-----
trunk/pge/src/main/java/org/apache/oodt/cas/pge/PGETaskInstance.java 1302224
trunk/pge/src/main/java/org/apache/oodt/cas/pge/logging/PgeLogHandler.java 1302224
trunk/pge/src/main/java/org/apache/oodt/cas/pge/logging/PgeLogRecord.java 1302224
trunk/pge/src/test/org/apache/oodt/cas/pge/TestPGETaskInstance.java 1302224
trunk/pge/src/test/org/apache/oodt/cas/pge/logging/TestPgeLogHandler.java 1302224
Diff: https://reviews.apache.org/r/4398/diff
Testing
-------
updated/added unit-tests
Thanks,
brian
> Port LoggerOutputStream in CAS-PGE to existing LoggerOutputStream in oodt-commons io package
> --------------------------------------------------------------------------------------------
>
> Key: OODT-423
> URL: https://issues.apache.org/jira/browse/OODT-423
> Project: OODT
> Issue Type: Sub-task
> Components: commons, pge wrapper framework
> Affects Versions: 0.3
> Environment: none
> Reporter: Brian Foster
> Assignee: Brian Foster
> Priority: Minor
> Fix For: 0.4
>
> Attachments: OODT-423.2012-03-18.cas-pge.patch.txt, OODT-423.2012-03-18.oodt-commons.patch.txt
>
>
> This will include porting unit-tests as well from CAS-PGE... CAS-PGE will be switched back to using oodt-commons LoggerOutputStream as well in this bug... CAS-PGE's LoggerOutputStream will be deleted.
--
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
[jira] [Commented] (OODT-423) Port LoggerOutputStream in CAS-PGE to
existing LoggerOutputStream in oodt-commons io package
Posted by "jiraposter@reviews.apache.org (Commented) (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OODT-423?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13232377#comment-13232377 ]
jiraposter@reviews.apache.org commented on OODT-423:
----------------------------------------------------
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4398/
-----------------------------------------------------------
(Updated 2012-03-18 21:25:45.978379)
Review request for oodt, Chris Mattmann, Ricky Nguyen, Paul Ramirez, and Thomas Bennett.
Summary (updated)
-------
This is the CAS-PGE portion of this change
This addresses bug OODT-423.
https://issues.apache.org/jira/browse/OODT-423
Diffs
-----
trunk/pge/src/main/java/org/apache/oodt/cas/pge/PGETaskInstance.java 1302224
trunk/pge/src/main/java/org/apache/oodt/cas/pge/logging/PgeLogHandler.java 1302224
trunk/pge/src/main/java/org/apache/oodt/cas/pge/logging/PgeLogRecord.java 1302224
trunk/pge/src/test/org/apache/oodt/cas/pge/TestPGETaskInstance.java 1302224
trunk/pge/src/test/org/apache/oodt/cas/pge/logging/TestPgeLogHandler.java 1302224
Diff: https://reviews.apache.org/r/4398/diff
Testing
-------
updated/added unit-tests
Thanks,
brian
> Port LoggerOutputStream in CAS-PGE to existing LoggerOutputStream in oodt-commons io package
> --------------------------------------------------------------------------------------------
>
> Key: OODT-423
> URL: https://issues.apache.org/jira/browse/OODT-423
> Project: OODT
> Issue Type: Sub-task
> Components: commons, pge wrapper framework
> Affects Versions: 0.3
> Environment: none
> Reporter: Brian Foster
> Assignee: Brian Foster
> Priority: Minor
> Fix For: 0.4
>
> Attachments: OODT-423.2012-03-18.cas-pge.patch.txt, OODT-423.2012-03-18.oodt-commons.patch.txt
>
>
> This will include porting unit-tests as well from CAS-PGE... CAS-PGE will be switched back to using oodt-commons LoggerOutputStream as well in this bug... CAS-PGE's LoggerOutputStream will be deleted.
--
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
[jira] [Commented] (OODT-423) Port LoggerOutputStream in CAS-PGE to
existing LoggerOutputStream in oodt-commons io package
Posted by "Brian Foster (Commented) (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OODT-423?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13232374#comment-13232374 ]
Brian Foster commented on OODT-423:
-----------------------------------
The issued can be review here:
- https://reviews.apache.org/r/4399/ (oodt-commons changes)
- https://reviews.apache.org/r/4398/ (cas-pge changes)
> Port LoggerOutputStream in CAS-PGE to existing LoggerOutputStream in oodt-commons io package
> --------------------------------------------------------------------------------------------
>
> Key: OODT-423
> URL: https://issues.apache.org/jira/browse/OODT-423
> Project: OODT
> Issue Type: Sub-task
> Components: commons, pge wrapper framework
> Affects Versions: 0.3
> Environment: none
> Reporter: Brian Foster
> Assignee: Brian Foster
> Priority: Minor
> Fix For: 0.4
>
> Attachments: OODT-423.2012-03-18.cas-pge.patch.txt, OODT-423.2012-03-18.oodt-commons.patch.txt
>
>
> This will include porting unit-tests as well from CAS-PGE... CAS-PGE will be switched back to using oodt-commons LoggerOutputStream as well in this bug... CAS-PGE's LoggerOutputStream will be deleted.
--
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
[jira] [Commented] (OODT-423) Port LoggerOutputStream in CAS-PGE to
existing LoggerOutputStream in oodt-commons io package
Posted by "jiraposter@reviews.apache.org (Commented) (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OODT-423?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13232672#comment-13232672 ]
jiraposter@reviews.apache.org commented on OODT-423:
----------------------------------------------------
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4399/#review6073
-----------------------------------------------------------
Ship it!
LGTM
- Chris
On 2012-03-18 21:26:02, brian Foster wrote:
bq.
bq. -----------------------------------------------------------
bq. This is an automatically generated e-mail. To reply, visit:
bq. https://reviews.apache.org/r/4399/
bq. -----------------------------------------------------------
bq.
bq. (Updated 2012-03-18 21:26:02)
bq.
bq.
bq. Review request for oodt, Chris Mattmann, Ricky Nguyen, Paul Ramirez, and Thomas Bennett.
bq.
bq.
bq. Summary
bq. -------
bq.
bq. This is the OODT-Commons portion of this change
bq.
bq.
bq. This addresses bug OODT-423.
bq. https://issues.apache.org/jira/browse/OODT-423
bq.
bq.
bq. Diffs
bq. -----
bq.
bq. trunk/commons/src/main/java/org/apache/oodt/commons/exec/ExecUtils.java 1302040
bq. trunk/commons/src/main/java/org/apache/oodt/commons/io/LoggerOutputStream.java 1302040
bq. trunk/commons/src/test/org/apache/oodt/commons/io/TestLoggerOutputStream.java PRE-CREATION
bq.
bq. Diff: https://reviews.apache.org/r/4399/diff
bq.
bq.
bq. Testing
bq. -------
bq.
bq. Updated/Added unit-tests
bq.
bq.
bq. Thanks,
bq.
bq. brian
bq.
bq.
> Port LoggerOutputStream in CAS-PGE to existing LoggerOutputStream in oodt-commons io package
> --------------------------------------------------------------------------------------------
>
> Key: OODT-423
> URL: https://issues.apache.org/jira/browse/OODT-423
> Project: OODT
> Issue Type: Sub-task
> Components: commons, pge wrapper framework
> Affects Versions: 0.3
> Environment: none
> Reporter: Brian Foster
> Assignee: Brian Foster
> Priority: Minor
> Fix For: 0.4
>
> Attachments: OODT-423.2012-03-18.cas-pge.patch.txt, OODT-423.2012-03-18.oodt-commons.patch.txt
>
>
> This will include porting unit-tests as well from CAS-PGE... CAS-PGE will be switched back to using oodt-commons LoggerOutputStream as well in this bug... CAS-PGE's LoggerOutputStream will be deleted.
--
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
[jira] [Commented] (OODT-423) Port LoggerOutputStream in CAS-PGE to
existing LoggerOutputStream in oodt-commons io package
Posted by "jiraposter@reviews.apache.org (Commented) (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OODT-423?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13232372#comment-13232372 ]
jiraposter@reviews.apache.org commented on OODT-423:
----------------------------------------------------
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4399/
-----------------------------------------------------------
Review request for oodt, Chris Mattmann, Ricky Nguyen, Paul Ramirez, and Thomas Bennett.
Summary
-------
This is the OODT-Commons portion of this change
This addresses bug OODT-423.
https://issues.apache.org/jira/browse/OODT-423
Diffs
-----
trunk/commons/src/main/java/org/apache/oodt/commons/exec/ExecUtils.java 1302040
trunk/commons/src/main/java/org/apache/oodt/commons/io/LoggerOutputStream.java 1302040
trunk/commons/src/test/org/apache/oodt/commons/io/TestLoggerOutputStream.java PRE-CREATION
Diff: https://reviews.apache.org/r/4399/diff
Testing
-------
Updated/Added unit-tests
Thanks,
brian
> Port LoggerOutputStream in CAS-PGE to existing LoggerOutputStream in oodt-commons io package
> --------------------------------------------------------------------------------------------
>
> Key: OODT-423
> URL: https://issues.apache.org/jira/browse/OODT-423
> Project: OODT
> Issue Type: Sub-task
> Components: commons, pge wrapper framework
> Affects Versions: 0.3
> Environment: none
> Reporter: Brian Foster
> Assignee: Brian Foster
> Priority: Minor
> Fix For: 0.4
>
> Attachments: OODT-423.2012-03-18.cas-pge.patch.txt, OODT-423.2012-03-18.oodt-commons.patch.txt
>
>
> This will include porting unit-tests as well from CAS-PGE... CAS-PGE will be switched back to using oodt-commons LoggerOutputStream as well in this bug... CAS-PGE's LoggerOutputStream will be deleted.
--
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
[jira] [Updated] (OODT-423) Port LoggerOutputStream in CAS-PGE to
existing LoggerOutputStream in oodt-commons io package
Posted by "Brian Foster (Updated) (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OODT-423?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Brian Foster updated OODT-423:
------------------------------
Attachment: OODT-423.2012-03-18.oodt-commons.patch.txt
OODT-423.2012-03-18.cas-pge.patch.txt
- attached patches
> Port LoggerOutputStream in CAS-PGE to existing LoggerOutputStream in oodt-commons io package
> --------------------------------------------------------------------------------------------
>
> Key: OODT-423
> URL: https://issues.apache.org/jira/browse/OODT-423
> Project: OODT
> Issue Type: Sub-task
> Components: commons, pge wrapper framework
> Affects Versions: 0.3
> Environment: none
> Reporter: Brian Foster
> Assignee: Brian Foster
> Priority: Minor
> Fix For: 0.4
>
> Attachments: OODT-423.2012-03-18.cas-pge.patch.txt, OODT-423.2012-03-18.oodt-commons.patch.txt
>
>
> This will include porting unit-tests as well from CAS-PGE... CAS-PGE will be switched back to using oodt-commons LoggerOutputStream as well in this bug... CAS-PGE's LoggerOutputStream will be deleted.
--
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
[jira] [Commented] (OODT-423) Port LoggerOutputStream in CAS-PGE to
existing LoggerOutputStream in oodt-commons io package
Posted by "jiraposter@reviews.apache.org (Commented) (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OODT-423?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13232669#comment-13232669 ]
jiraposter@reviews.apache.org commented on OODT-423:
----------------------------------------------------
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4398/#review6072
-----------------------------------------------------------
Ship it!
LGTM!
- Chris
On 2012-03-18 21:25:45, brian Foster wrote:
bq.
bq. -----------------------------------------------------------
bq. This is an automatically generated e-mail. To reply, visit:
bq. https://reviews.apache.org/r/4398/
bq. -----------------------------------------------------------
bq.
bq. (Updated 2012-03-18 21:25:45)
bq.
bq.
bq. Review request for oodt, Chris Mattmann, Ricky Nguyen, Paul Ramirez, and Thomas Bennett.
bq.
bq.
bq. Summary
bq. -------
bq.
bq. This is the CAS-PGE portion of this change
bq.
bq.
bq. This addresses bug OODT-423.
bq. https://issues.apache.org/jira/browse/OODT-423
bq.
bq.
bq. Diffs
bq. -----
bq.
bq. trunk/pge/src/main/java/org/apache/oodt/cas/pge/PGETaskInstance.java 1302224
bq. trunk/pge/src/main/java/org/apache/oodt/cas/pge/logging/PgeLogHandler.java 1302224
bq. trunk/pge/src/main/java/org/apache/oodt/cas/pge/logging/PgeLogRecord.java 1302224
bq. trunk/pge/src/test/org/apache/oodt/cas/pge/TestPGETaskInstance.java 1302224
bq. trunk/pge/src/test/org/apache/oodt/cas/pge/logging/TestPgeLogHandler.java 1302224
bq.
bq. Diff: https://reviews.apache.org/r/4398/diff
bq.
bq.
bq. Testing
bq. -------
bq.
bq. updated/added unit-tests
bq.
bq.
bq. Thanks,
bq.
bq. brian
bq.
bq.
> Port LoggerOutputStream in CAS-PGE to existing LoggerOutputStream in oodt-commons io package
> --------------------------------------------------------------------------------------------
>
> Key: OODT-423
> URL: https://issues.apache.org/jira/browse/OODT-423
> Project: OODT
> Issue Type: Sub-task
> Components: commons, pge wrapper framework
> Affects Versions: 0.3
> Environment: none
> Reporter: Brian Foster
> Assignee: Brian Foster
> Priority: Minor
> Fix For: 0.4
>
> Attachments: OODT-423.2012-03-18.cas-pge.patch.txt, OODT-423.2012-03-18.oodt-commons.patch.txt
>
>
> This will include porting unit-tests as well from CAS-PGE... CAS-PGE will be switched back to using oodt-commons LoggerOutputStream as well in this bug... CAS-PGE's LoggerOutputStream will be deleted.
--
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
[jira] [Resolved] (OODT-423) Port LoggerOutputStream in CAS-PGE to
existing LoggerOutputStream in oodt-commons io package
Posted by "Brian Foster (Resolved) (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OODT-423?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Brian Foster resolved OODT-423.
-------------------------------
Resolution: Fixed
- fixed in r1302592 and r1302593
> Port LoggerOutputStream in CAS-PGE to existing LoggerOutputStream in oodt-commons io package
> --------------------------------------------------------------------------------------------
>
> Key: OODT-423
> URL: https://issues.apache.org/jira/browse/OODT-423
> Project: OODT
> Issue Type: Sub-task
> Components: commons, pge wrapper framework
> Affects Versions: 0.3
> Environment: none
> Reporter: Brian Foster
> Assignee: Brian Foster
> Priority: Minor
> Fix For: 0.4
>
> Attachments: OODT-423.2012-03-18.cas-pge.patch.txt, OODT-423.2012-03-18.oodt-commons.patch.txt
>
>
> This will include porting unit-tests as well from CAS-PGE... CAS-PGE will be switched back to using oodt-commons LoggerOutputStream as well in this bug... CAS-PGE's LoggerOutputStream will be deleted.
--
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
[jira] [Commented] (OODT-423) Port LoggerOutputStream in CAS-PGE to
existing LoggerOutputStream in oodt-commons io package
Posted by "jiraposter@reviews.apache.org (Commented) (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/OODT-423?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13232376#comment-13232376 ]
jiraposter@reviews.apache.org commented on OODT-423:
----------------------------------------------------
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4399/
-----------------------------------------------------------
(Updated 2012-03-18 21:26:02.155994)
Review request for oodt, Chris Mattmann, Ricky Nguyen, Paul Ramirez, and Thomas Bennett.
Summary (updated)
-------
This is the OODT-Commons portion of this change
This addresses bug OODT-423.
https://issues.apache.org/jira/browse/OODT-423
Diffs
-----
trunk/commons/src/main/java/org/apache/oodt/commons/exec/ExecUtils.java 1302040
trunk/commons/src/main/java/org/apache/oodt/commons/io/LoggerOutputStream.java 1302040
trunk/commons/src/test/org/apache/oodt/commons/io/TestLoggerOutputStream.java PRE-CREATION
Diff: https://reviews.apache.org/r/4399/diff
Testing
-------
Updated/Added unit-tests
Thanks,
brian
> Port LoggerOutputStream in CAS-PGE to existing LoggerOutputStream in oodt-commons io package
> --------------------------------------------------------------------------------------------
>
> Key: OODT-423
> URL: https://issues.apache.org/jira/browse/OODT-423
> Project: OODT
> Issue Type: Sub-task
> Components: commons, pge wrapper framework
> Affects Versions: 0.3
> Environment: none
> Reporter: Brian Foster
> Assignee: Brian Foster
> Priority: Minor
> Fix For: 0.4
>
> Attachments: OODT-423.2012-03-18.cas-pge.patch.txt, OODT-423.2012-03-18.oodt-commons.patch.txt
>
>
> This will include porting unit-tests as well from CAS-PGE... CAS-PGE will be switched back to using oodt-commons LoggerOutputStream as well in this bug... CAS-PGE's LoggerOutputStream will be deleted.
--
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