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 14626: OODT 562 - Exposing parent-child & element addition/deletion functionality for product types using cas-curator
Date Sat, 08 Nov 2014 17:02:48 GMT

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



trunk/curator/src/main/java/org/apache/oodt/cas/curation/service/MetadataResource.java
<https://reviews.apache.org/r/14626/#comment101883>

    I would prefer just "/parentmap" (no need for GET, implied by the HTTP protocol and METHOD).



trunk/curator/src/main/java/org/apache/oodt/cas/curation/service/MetadataResource.java
<https://reviews.apache.org/r/14626/#comment101884>

    I would change this to "/parent/add"



trunk/curator/src/main/java/org/apache/oodt/cas/curation/service/MetadataResource.java
<https://reviews.apache.org/r/14626/#comment101885>

    "/parent/remove"



trunk/curator/src/main/java/org/apache/oodt/cas/curation/service/MetadataResource.java
<https://reviews.apache.org/r/14626/#comment101886>

    "/elements/add"



trunk/curator/src/main/java/org/apache/oodt/cas/curation/service/MetadataResource.java
<https://reviews.apache.org/r/14626/#comment101887>

    "/elements"



trunk/curator/src/main/java/org/apache/oodt/cas/curation/service/MetadataResource.java
<https://reviews.apache.org/r/14626/#comment101888>

    "/elements/remove" (and if you want to remove a specific one, do "/elements/remove/{elementid}"



trunk/curator/src/main/java/org/apache/oodt/cas/curation/service/MetadataResource.java
<https://reviews.apache.org/r/14626/#comment101889>

    "/elements/remove/{list of ids, comma separated?}"



trunk/curator/src/main/java/org/apache/oodt/cas/curation/service/MetadataResource.java
<https://reviews.apache.org/r/14626/#comment101890>

    "/typeswithelement/{elementid}"


- Chris Mattmann


On Nov. 7, 2014, 10:13 p.m., Varun Ratnakar wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/14626/
> -----------------------------------------------------------
> 
> (Updated Nov. 7, 2014, 10:13 p.m.)
> 
> 
> Review request for oodt and Chris Mattmann.
> 
> 
> Bugs: OODT-562
>     https://issues.apache.org/jira/browse/OODT-562
> 
> 
> Repository: oodt
> 
> 
> Description
> -------
> 
> This patch is for https://issues.apache.org/jira/browse/OODT-562
> 
> 1. It adds some functions to the cas-curator's Metadata service
> 2. Added functions to the file-manager in XMLValidationLayer for adding/removing/retrieving
parents of product types)
> 
> 
> Diffs
> -----
> 
>   trunk/curator/src/main/java/org/apache/oodt/cas/curation/service/MetadataResource.java
1636677 
>   trunk/filemgr/src/main/java/org/apache/oodt/cas/filemgr/util/XmlStructFactory.java
1636677 
>   trunk/filemgr/src/main/java/org/apache/oodt/cas/filemgr/validation/XMLValidationLayer.java
1636677 
> 
> Diff: https://reviews.apache.org/r/14626/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Varun Ratnakar
> 
>


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