Oh Ok. Got it. It makes sense now. I thought the ProductType ID was a key constraint for Products table too. I checked the SQL files and realized that disabling or removing the Product Types wouldn't create a problem here.
Thanks! On Sat, Apr 2, 2011 at 6:38 PM, Mattmann, Chris A (388J) < chris.a.mattm...@jpl.nasa.gov> wrote: > Hi Sheryl, > > I think the thinking there was that a user may in fact use > "removeProductType" to simply "disable" a product type, but may later > somehow want to undo that. > > Ideally the function should have an option as to whether or not to actual > remove the met and refs tables, but that's not there, yet... ^_^ Patches > welcome! > > Cheers, > Chris > > On Apr 2, 2011, at 4:26 PM, Sheryl John wrote: > > > Hi, > > > > I am curious about the removeProductType() method in the following class > : > > > https://svn.apache.org/repos/asf/oodt/trunk/filemgr/src/main/java/org/apache/oodt/cas/filemgr/repository/DataSourceRepositoryManager.java > > > > < > https://svn.apache.org/repos/asf/oodt/trunk/filemgr/src/main/java/org/apache/oodt/cas/filemgr/repository/DataSourceRepositoryManager.java > >Since > > each Product is a member of a single Product Type, don't we have to > delete > > all the Products of the deleted Product Types too (i..e reference and > > metadata table.) ? > > > > I read the TODO comment and I am just curious as to why the deletion was > not > > done here or rather, the reason behind the assumption in the comment. > > > > > > Thanks, > > -- > > Sheryl > > > ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ > 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 > ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ > > -- Sheryl