On Wed, Jan 27, 2010 at 10:16 AM, Simon Brown <st...@cam.ac.uk> wrote:
> I confess that the reason why we are having this discussion at all > eludes me. It seems like a fairly obvious bug for the importer to > prune the indexes so many times (the comment for pruneIndexes() even > says "called from the public interfaces or at the end of a batch > indexing process"), it has a demonstrably detrimental effect on the > performance of the software, and the fix for it is not particularly > complicated. Is there something I've missed? > > -- > Simon Brown <st...@cam.ac.uk> - Cambridge University Computing Service > +44 1223 3 34714 - New Museums Site, Pembroke Street, Cambridge CB2 3QH We discuss it because we seek to maintain an appropriate separation of concerns in our architecture. And because Graham usually challenges us to look at aspects of that architecture that are important. What is under discussion is not that performance can't be improved by your patch, you've identified a very important issue in batch processing. We are discussing architecturally if we want to alter the Context/EventManager framework and expose calls to pruneIndex. We want to be careful to avoid exposing too much of the internals of the Browse system outside in the application architecture. Excellent work on finding a means to improve DSpace performance. Cheers, Mark -- Mark R. Diggory Head of U.S. Operations - @mire http://www.atmire.com - Institutional Repository Solutions http://www.togather.eu - Before getting together, get t...@ther ------------------------------------------------------------------------------ The Planet: dedicated and managed hosting, cloud storage, colocation Stay online with enterprise data centers and the best network in the business Choose flexible plans and management services without long-term contracts Personal 24x7 support from experience hosting pros just a phone call away. http://p.sf.net/sfu/theplanet-com _______________________________________________ Dspace-devel mailing list Dspace-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/dspace-devel
