Hey Chris, Yep, you're right so I'll look into this further. I know that commit broke the build as Jenkins keeps telling the list. I wonder if the bug was there before. If I recall you just expanded the unit test. Anyhow when I have more time I'll try to track this down if someone doesn't get to it first. Essentially, the unit test is failing right now.
--Paul On Mar 21, 2012, at 1:18 PM, Mattmann, Chris A (388J) wrote: > Hi Paul, > > On Mar 21, 2012, at 3:41 PM, Ramirez, Paul M (388J) wrote: > >> Hey Guys, >> >> This commit broke the unit test >> (org.apache.oodt.cas.filemgr.catalog.TestMappedDataSourceCatalog) with the >> addition of: >> >> assertEquals("ShouldBeFirstForPage.txt", >> ((Product)myCat.getFirstPage(type).getPageProducts().get(0)).getProductName()); > > Hmmm, that commit was from me. Read on below. > >> >> The data source catalog is sorted by product id so this test does not >> succeed (since 20 products were added before it) as it would with the lucene >> catalog which is naturally sorted by CAS.ProductReceivedTime. I can fix the >> test to look at the second page but wanted to make sure the intent was >> captured. Also this is sort of an oddity with our catalogs having one set >> sorted by default with product id and the other with time. >> > > Sorting in the DataSourceCatalog by product ID should be equivalent to > sorting in the Lucene Catalog by ProductReceivedTime, right? > > Cheers, > Chris > > ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ > Chris Mattmann, Ph.D. > Senior Computer Scientist > NASA Jet Propulsion Laboratory Pasadena, CA 91109 USA > Office: 171-266B, Mailstop: 171-246 > Email: chris.a.mattm...@nasa.gov > WWW: http://sunset.usc.edu/~mattmann/ > ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ > Adjunct Assistant Professor, Computer Science Department > University of Southern California, Los Angeles, CA 90089 USA > ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ >