----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8616/#review14542 -----------------------------------------------------------
./trunk/filemgr/src/main/java/org/apache/oodt/cas/filemgr/catalog/DataSourceCatalog.java <https://reviews.apache.org/r/8616/#comment30944> Changes like this I don't get -- even if they are auto generated by Eclipse -- they're extraneous since they are just moved higher up in the file. ./trunk/filemgr/src/main/java/org/apache/oodt/cas/filemgr/catalog/DataSourceCatalog.java <https://reviews.apache.org/r/8616/#comment30945> can we change quoteit to be quoteIt (camel case, Java style?) ./trunk/filemgr/src/main/java/org/apache/oodt/cas/filemgr/catalog/DataSourceCatalog.java <https://reviews.apache.org/r/8616/#comment30946> It would be great to change this to quoteIt. ./trunk/filemgr/src/main/java/org/apache/oodt/cas/filemgr/catalog/LenientDataSourceCatalog.java <https://reviews.apache.org/r/8616/#comment30947> why is this a FIXME? - Chris Mattmann On Dec. 15, 2012, 7 p.m., Chris Mattmann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/8616/ > ----------------------------------------------------------- > > (Updated Dec. 15, 2012, 7 p.m.) > > > Review request for oodt, brian Foster, Ricky Nguyen, Paul Ramirez, Thomas > Bennett, and Cameron Goodale. > > > Description > ------- > > Patch from Luca Cinquini and I to implement support for UUID based Data > Source Catalogs. > > > This addresses bug OODT-544. > https://issues.apache.org/jira/browse/OODT-544 > > > Diffs > ----- > > > ./trunk/filemgr/src/main/java/org/apache/oodt/cas/filemgr/catalog/DataSourceCatalog.java > 1422312 > > ./trunk/filemgr/src/main/java/org/apache/oodt/cas/filemgr/catalog/DataSourceCatalogFactory.java > 1422312 > > ./trunk/filemgr/src/main/java/org/apache/oodt/cas/filemgr/catalog/LenientDataSourceCatalog.java > PRE-CREATION > > ./trunk/filemgr/src/main/java/org/apache/oodt/cas/filemgr/util/DbStructFactory.java > 1422312 > > Diff: https://reviews.apache.org/r/8616/diff/ > > > Testing > ------- > > We have tested this on my transient detection project for V-FASTR. > > > Thanks, > > Chris Mattmann > >
