----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14513/#review27055 -----------------------------------------------------------
trunk/webapp/fmprod/pom.xml <https://reviews.apache.org/r/14513/#comment52682> may want to upgrade this to 1.4 but not a blocker - Chris Mattmann On Oct. 6, 2013, 10:37 a.m., Ross Laidlaw wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/14513/ > ----------------------------------------------------------- > > (Updated Oct. 6, 2013, 10:37 a.m.) > > > Review request for oodt, Chris Mattmann and Rishi Verma. > > > Bugs: OODT-612 and OODT-613 > https://issues.apache.org/jira/browse/OODT-612 > https://issues.apache.org/jira/browse/OODT-613 > > > Repository: oodt > > > Description > ------- > > This patch aims to address issues OODT-612 and OODT-613. Please see the JIRA > issues for discussions on the chosen designs/implementations. > > The Java code for this patch is contained in a new package > 'org.apache.oodt.cas.product.jaxrs'. This package is intended to replace the > 'org.apache.oodt.cas.product.service' package that was committed for the GSoC > (Google Summer of Code 2013) midterm, i.e. the cas.product.service package > will be deleted and replaced with this reworked design. > > Documentation for this work has been added to the OODT wiki as detailed on > OODT-614. > > Work is ongoing for this package and many improvements can be made, for > example those mentioned in the 'development ideas' section at the bottom of > the developer guide. > > > Diffs > ----- > > trunk/webapp/fmprod/pom.xml 1517156 > > trunk/webapp/fmprod/src/main/java/org/apache/oodt/cas/product/jaxrs/configurations/RdfConfiguration.java > PRE-CREATION > > trunk/webapp/fmprod/src/main/java/org/apache/oodt/cas/product/jaxrs/configurations/RssConfiguration.java > PRE-CREATION > > trunk/webapp/fmprod/src/main/java/org/apache/oodt/cas/product/jaxrs/configurations/RssNamespace.java > PRE-CREATION > > trunk/webapp/fmprod/src/main/java/org/apache/oodt/cas/product/jaxrs/configurations/RssTag.java > PRE-CREATION > > trunk/webapp/fmprod/src/main/java/org/apache/oodt/cas/product/jaxrs/configurations/RssTagAttribute.java > PRE-CREATION > > trunk/webapp/fmprod/src/main/java/org/apache/oodt/cas/product/jaxrs/configurations/package-info.java > PRE-CREATION > > trunk/webapp/fmprod/src/main/java/org/apache/oodt/cas/product/jaxrs/exceptions/BadRequestException.java > PRE-CREATION > > trunk/webapp/fmprod/src/main/java/org/apache/oodt/cas/product/jaxrs/exceptions/InternalServerErrorException.java > PRE-CREATION > > trunk/webapp/fmprod/src/main/java/org/apache/oodt/cas/product/jaxrs/exceptions/NotFoundException.java > PRE-CREATION > > trunk/webapp/fmprod/src/main/java/org/apache/oodt/cas/product/jaxrs/exceptions/package-info.java > PRE-CREATION > > trunk/webapp/fmprod/src/main/java/org/apache/oodt/cas/product/jaxrs/filters/BackwardsCompatibleInterceptor.java > PRE-CREATION > > trunk/webapp/fmprod/src/main/java/org/apache/oodt/cas/product/jaxrs/filters/package-info.java > PRE-CREATION > > trunk/webapp/fmprod/src/main/java/org/apache/oodt/cas/product/jaxrs/package-info.java > PRE-CREATION > > trunk/webapp/fmprod/src/main/java/org/apache/oodt/cas/product/jaxrs/resources/DatasetResource.java > PRE-CREATION > > trunk/webapp/fmprod/src/main/java/org/apache/oodt/cas/product/jaxrs/resources/MetadataResource.java > PRE-CREATION > > trunk/webapp/fmprod/src/main/java/org/apache/oodt/cas/product/jaxrs/resources/ProductResource.java > PRE-CREATION > > trunk/webapp/fmprod/src/main/java/org/apache/oodt/cas/product/jaxrs/resources/ReferenceResource.java > PRE-CREATION > > trunk/webapp/fmprod/src/main/java/org/apache/oodt/cas/product/jaxrs/resources/TransferResource.java > PRE-CREATION > > trunk/webapp/fmprod/src/main/java/org/apache/oodt/cas/product/jaxrs/resources/TransfersResource.java > PRE-CREATION > > trunk/webapp/fmprod/src/main/java/org/apache/oodt/cas/product/jaxrs/resources/package-info.java > PRE-CREATION > > trunk/webapp/fmprod/src/main/java/org/apache/oodt/cas/product/jaxrs/services/CasProductJaxrsService.java > PRE-CREATION > > trunk/webapp/fmprod/src/main/java/org/apache/oodt/cas/product/jaxrs/services/package-info.java > PRE-CREATION > > trunk/webapp/fmprod/src/main/java/org/apache/oodt/cas/product/jaxrs/servlets/CasProductJaxrsServlet.java > PRE-CREATION > > trunk/webapp/fmprod/src/main/java/org/apache/oodt/cas/product/jaxrs/servlets/package-info.java > PRE-CREATION > > trunk/webapp/fmprod/src/main/java/org/apache/oodt/cas/product/jaxrs/writers/DatasetRdfWriter.java > PRE-CREATION > > trunk/webapp/fmprod/src/main/java/org/apache/oodt/cas/product/jaxrs/writers/DatasetRssWriter.java > PRE-CREATION > > trunk/webapp/fmprod/src/main/java/org/apache/oodt/cas/product/jaxrs/writers/DatasetZipWriter.java > PRE-CREATION > > trunk/webapp/fmprod/src/main/java/org/apache/oodt/cas/product/jaxrs/writers/DatasetZipper.java > PRE-CREATION > > trunk/webapp/fmprod/src/main/java/org/apache/oodt/cas/product/jaxrs/writers/ProductRssWriter.java > PRE-CREATION > > trunk/webapp/fmprod/src/main/java/org/apache/oodt/cas/product/jaxrs/writers/ProductZipWriter.java > PRE-CREATION > > trunk/webapp/fmprod/src/main/java/org/apache/oodt/cas/product/jaxrs/writers/ProductZipper.java > PRE-CREATION > > trunk/webapp/fmprod/src/main/java/org/apache/oodt/cas/product/jaxrs/writers/RdfWriter.java > PRE-CREATION > > trunk/webapp/fmprod/src/main/java/org/apache/oodt/cas/product/jaxrs/writers/ReferenceFileWriter.java > PRE-CREATION > > trunk/webapp/fmprod/src/main/java/org/apache/oodt/cas/product/jaxrs/writers/ReferenceRssWriter.java > PRE-CREATION > > trunk/webapp/fmprod/src/main/java/org/apache/oodt/cas/product/jaxrs/writers/ReferenceZipWriter.java > PRE-CREATION > > trunk/webapp/fmprod/src/main/java/org/apache/oodt/cas/product/jaxrs/writers/RssWriter.java > PRE-CREATION > > trunk/webapp/fmprod/src/main/java/org/apache/oodt/cas/product/jaxrs/writers/TransfersRssWriter.java > PRE-CREATION > > trunk/webapp/fmprod/src/main/java/org/apache/oodt/cas/product/jaxrs/writers/package-info.java > PRE-CREATION > trunk/webapp/fmprod/src/main/resources/georss-config.xml 1517156 > trunk/webapp/fmprod/src/main/resources/logging.properties 1517156 > trunk/webapp/fmprod/src/main/resources/rdfconf.xml 1517156 > trunk/webapp/fmprod/src/main/resources/rss-transfer-conf.xml 1517156 > trunk/webapp/fmprod/src/main/resources/rssconf.xml 1517156 > trunk/webapp/fmprod/src/main/webapp/META-INF/context.xml 1517156 > trunk/webapp/fmprod/src/main/webapp/WEB-INF/web.xml 1517156 > > trunk/webapp/fmprod/src/test/java/org/apache/oodt/cas/product/jaxrs/resources/DatasetResourceTest.java > PRE-CREATION > > trunk/webapp/fmprod/src/test/java/org/apache/oodt/cas/product/jaxrs/resources/MetadataResourceTest.java > PRE-CREATION > > trunk/webapp/fmprod/src/test/java/org/apache/oodt/cas/product/jaxrs/resources/ProductResourceTest.java > PRE-CREATION > > trunk/webapp/fmprod/src/test/java/org/apache/oodt/cas/product/jaxrs/resources/ReferenceResourceTest.java > PRE-CREATION > > trunk/webapp/fmprod/src/test/java/org/apache/oodt/cas/product/jaxrs/resources/TransferResourceTest.java > PRE-CREATION > > trunk/webapp/fmprod/src/test/java/org/apache/oodt/cas/product/jaxrs/resources/TransfersResourceTest.java > PRE-CREATION > trunk/webapp/fmprod/src/test/resources/test.logging.properties 1517156 > > Diff: https://reviews.apache.org/r/14513/diff/ > > > Testing > ------- > > I implemented several JUnit tests for the new package. Some of these unit > tests are included in this patch, while others have yet to be formalised but > will be presented in future patches. > > I also performed a lot of manual testing by deploying the web application and > testing sets of URLs via web browsers and command line tools (e.g. curl). I > checked all of the outputs for validity (e.g. as valid XML, RSS, RDF, etc). > > > Thanks, > > Ross Laidlaw > >