Re: Review Request: Improved CAS-PGE logging, product ingest fail CAS-PGE will fail, and PgeConfigBuilder configurable via PgeTaskMetKeys

2012-03-17 Thread brian Foster
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4393/ --- (Updated 2012-03-17 20:19:38.958054) Review request for oodt, Chris Mattmann, Ri

Re: Review Request: Improved CAS-PGE logging, product ingest fail CAS-PGE will fail, and PgeConfigBuilder configurable via PgeTaskMetKeys

2012-03-17 Thread brian Foster
> On 2012-03-17 02:35:38, Chris Mattmann wrote: > > trunk/pge/src/main/java/org/apache/oodt/cas/pge/PGETaskInstance.java, line > > 133 > > > > > > Typo, should be //Run the PGE and process its data. done! > On 2012-03

Re: Review Request: Improved CAS-PGE logging, product ingest fail CAS-PGE will fail, and PgeConfigBuilder configurable via PgeTaskMetKeys

2012-03-17 Thread Chris Mattmann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4393/#review6065 --- Ship it! LGTM, thanks! - Chris On 2012-03-17 20:19:38, brian Foste

Review Request: Improved CAS-PGE logging, product ingest fail CAS-PGE will fail, and PgeConfigBuilder configurable via PgeTaskMetKeys

2012-03-16 Thread brian Foster
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4393/ --- Review request for oodt, Chris Mattmann, Ricky Nguyen, Paul Ramirez, and Thomas B

Re: Review Request: Improved CAS-PGE logging, product ingest fail CAS-PGE will fail, and PgeConfigBuilder configurable via PgeTaskMetKeys

2012-03-16 Thread Chris Mattmann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4393/#review6060 --- trunk/pge/src/main/java/org/apache/oodt/cas/pge/PGETaskInstance.java

Re: Review Request: Improved CAS-PGE logging, product ingest fail CAS-PGE will fail, and PgeConfigBuilder configurable via PgeTaskMetKeys

2012-03-16 Thread brian Foster
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4393/ --- (Updated 2012-03-16 23:32:12.090230) Review request for oodt, Chris Mattmann, Ri