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

Review request for oodt.


Description
-------

- Updated XmlFilePgeConfigBuilder with <fileStaging> tag support
- Reorganize XmlFilePgeConfigBuilder in the process to clean it up


Diffs
-----

  trunk/pge/src/main/java/org/apache/oodt/cas/pge/config/FileStagingInfo.java 
1332872 
  trunk/pge/src/main/java/org/apache/oodt/cas/pge/config/PgeConfigMetKeys.java 
1311492 
  
trunk/pge/src/main/java/org/apache/oodt/cas/pge/config/XmlFilePgeConfigBuilder.java
 1311492 
  trunk/pge/src/main/java/org/apache/oodt/cas/pge/metadata/PgeTaskMetKeys.java 
1332872 
  trunk/pge/src/main/java/org/apache/oodt/cas/pge/util/Pair.java PRE-CREATION 
  trunk/pge/src/main/java/org/apache/oodt/cas/pge/util/XmlHelper.java 
PRE-CREATION 
  
trunk/pge/src/main/java/org/apache/oodt/cas/pge/writers/CsvConfigFileWriter.java
 PRE-CREATION 
  
trunk/pge/src/main/java/org/apache/oodt/cas/pge/writers/TextConfigFileWriter.java
 PRE-CREATION 
  trunk/pge/src/main/testdata/blank-pge-config.xml PRE-CREATION 
  trunk/pge/src/main/testdata/common-metadata-pge-config.xml PRE-CREATION 
  trunk/pge/src/main/testdata/custom-metadata-only-pge-config.xml PRE-CREATION 
  trunk/pge/src/main/testdata/data-file-1.txt PRE-CREATION 
  trunk/pge/src/main/testdata/data-file-2.txt PRE-CREATION 
  trunk/pge/src/main/testdata/dyn-input-writers-pge-config.xml PRE-CREATION 
  trunk/pge/src/main/testdata/imports-only-pge-config.xml PRE-CREATION 
  trunk/pge/src/main/testdata/pge-config.xml PRE-CREATION 
  trunk/pge/src/test/org/apache/oodt/cas/pge/TestPGETaskInstance.java 1311492 
  
trunk/pge/src/test/org/apache/oodt/cas/pge/config/TestXmlFilePgeConfigBuilder.java
 PRE-CREATION 
  trunk/pge/src/test/org/apache/oodt/cas/pge/util/TestXmlHelper.java 
PRE-CREATION 

Diff: https://reviews.apache.org/r/5382/diff/


Testing
-------

Wrote unit-tests


Thanks,

brian Foster

Reply via email to