oodt-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Chris Mattmann" <mattm...@apache.org>
Subject Re: Review Request: Updates from branch cas-pge
Date Sat, 10 Mar 2012 17:39:26 GMT

-----------------------------------------------------------
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 compat with the existing
PCS_ keys and not *force* folks to have to upgrade to them (which would force them to update
the configs). One idea I had for doing this was to maintain like 2 interfaces, one for v1-keys
(the "_" delimited ones), and another for v2-keys (the "/" delimited ones), and then allow
folks using CAS-PGE to switch between them with a config file option in the pge-config.xml.
Thoughts? I can work this up if you are +1.

- Chris


On 2012-03-09 08:17:12, brian Foster wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/4196/
> -----------------------------------------------------------
> 
> (Updated 2012-03-09 08:17:12)
> 
> 
> Review request for oodt, Chris Mattmann and Paul Ramirez.
> 
> 
> Summary
> -------
> 
> This includes the changes from wengine-branch cas-pge... still needs to add backwards
compatibility for MetKeys (i.e. PCS_* etc)
> 
> 
> This addresses bug OODT-262.
>     https://issues.apache.org/jira/browse/OODT-262
> 
> 
> Diffs
> -----
> 
>   trunk/pge/src/main/java/org/apache/oodt/cas/pge/PGETaskInstance.java 1297147 
>   trunk/pge/src/main/java/org/apache/oodt/cas/pge/config/FileBasedPgeConfigBuilder.java
PRE-CREATION 
>   trunk/pge/src/main/java/org/apache/oodt/cas/pge/config/FileStagingInfo.java PRE-CREATION

>   trunk/pge/src/main/java/org/apache/oodt/cas/pge/config/PgeConfig.java 1297147 
>   trunk/pge/src/main/java/org/apache/oodt/cas/pge/config/PgeConfigBuilder.java 1297147

>   trunk/pge/src/main/java/org/apache/oodt/cas/pge/config/PgeConfigMetKeys.java 1297147

>   trunk/pge/src/main/java/org/apache/oodt/cas/pge/config/XmlFilePgeConfigBuilder.java
1297147 
>   trunk/pge/src/main/java/org/apache/oodt/cas/pge/metadata/PGETaskMetKeys.java PRE-CREATION

>   trunk/pge/src/main/java/org/apache/oodt/cas/pge/metadata/PgeMetadata.java 1297147 
>   trunk/pge/src/main/java/org/apache/oodt/cas/pge/staging/FileManagerFileStager.java
PRE-CREATION 
>   trunk/pge/src/main/java/org/apache/oodt/cas/pge/staging/FileStager.java PRE-CREATION

>   trunk/pge/src/main/java/org/apache/oodt/cas/pge/writers/PcsMetFileWriter.java 1297147

> 
> Diff: https://reviews.apache.org/r/4196/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> brian
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message