----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4374/#review6026 -----------------------------------------------------------
trunk/filemgr/src/main/java/org/apache/oodt/cas/filemgr/catalog/LuceneCatalog.java <https://reviews.apache.org/r/4374/#comment13010> What about: String element = elements.get(i); if (fullMetadata.containsKey(element) { reducedMetadata.addMetadata(element, fullMetadata.getAllMetadata(element); } also add addMetadata() really should be replaceMetadata()... although in this cause unless the same element is in the elements List more than once addMetadata() will work the same - brian On 2012-03-16 03:18:23, Ricky Nguyen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/4374/ > ----------------------------------------------------------- > > (Updated 2012-03-16 03:18:23) > > > Review request for oodt, Chris Mattmann, brian Foster, Paul Ramirez, Sheryl > John, and Thomas Bennett. > > > Summary > ------- > > filemgr query throws NPE when some products have undefined metadata values > > > This addresses bug OODT-413. > https://issues.apache.org/jira/browse/OODT-413 > > > Diffs > ----- > > > trunk/filemgr/src/main/java/org/apache/oodt/cas/filemgr/catalog/LuceneCatalog.java > 1301315 > > trunk/filemgr/src/test/org/apache/oodt/cas/filemgr/catalog/TestLuceneCatalog.java > 1301315 > > Diff: https://reviews.apache.org/r/4374/diff > > > Testing > ------- > > tested at CHLA with command: > > ./filemgr-client -u $FILEMGR_URL -op -sql -of '$VpsEpisodeStartTime' -q > "SELECT VpsEpisodeStartTime FROM CernerEvents" > > > Thanks, > > Ricky > >
