Hey Luca,

I noticed that the SolrCatalog is in package
o.a.oodt.cas.filemgr.catalog.solr
instead of o.a.oodt.cas.filemgr.catalog, matching all other catalog
implementations.
I realize this is because you added specific classes to support the Solr
Catalog
and put them in that package as well. I think these classes should be
dispersed
either to util packages in the File Manager, or in some other support
structure
(e.g., as private internal classes to the SolrCatalog or
SolrCatalogFactory) so
as to keep consistent with the rest of the components. I'll make this
change in
the next few weeks (and file a JIRA issue) should there be no objections.

Thanks!

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
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++




Reply via email to