Re: Review Request: Ability for File Manager to stage an ingested Product to one of its clients

2012-03-10 Thread brian Foster
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4195/ --- (Updated 2012-03-09 07:55:23.448447) Review request for oodt, Chris Mattmann

Re: Review Request: Updates from branch cas-pge

2012-03-10 Thread Chris Mattmann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4196/#review5818 --- Hey Brian, one of the ideas I had on this was to maintain like back

Re: Review Request: Updates from branch cas-pge

2012-03-10 Thread brian Foster
On 2012-03-06 15:23:53, Paul Ramirez wrote: Looks good most of my comments are a request for some unit tests especially where stuff has been refactored and there is new code. Some of the stuff wasn't a change as the code was just reformatted. Hey paul thanks for the comments, also

Re: Review Request: Updates from branch cas-pge

2012-03-10 Thread brian Foster
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4196/ --- (Updated 2012-03-09 08:17:12.801020) Review request for oodt, Chris Mattmann

Re: Review Request: Port wengine ControlMetadata to cas-pge's PgeMetadata

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

Re: Review Request: Port wengine ControlMetadata to cas-pge's PgeMetadata

2012-03-10 Thread Chris Mattmann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4269/#review5821 --- Ship it! My comments are SUPER minor. This rocks. My +1 to ship it.