[PLIP-Advisories] Re: [Plone] #9352: combination of search results plips

plip-advisories at lists.plone.org plip-advisories at lists.plone.org
Tue Sep 1 04:39:15 UTC 2009

#9352: combination of search results plips
 Reporter:  laurenskling  |        Owner:  elvix   
     Type:  PLIP          |       Status:  assigned
 Priority:  minor         |    Milestone:  4.0     
Component:  Unknown       |   Resolution:          
 Keywords:                |  

Comment(by davisagli):

 I found an error in my initial ab benchmark.  I thought that I had
 convinced the old search page to not batch, but it was still doing so.
 Here are my results after temporarily modifying the old search.pt to not

  * old: 1.88 req/sec.
  * new: 1.17 req/sec. (e.g. 38% slower)

 However, in realistic cases search results will be batched.  Here are my
 results after reverting to the original search.pt, then modifying the
 IContentListing-based search view to slice its catalog results to 30
 before adapting to IContentListing (to simulate batching, since it is not
 supported yet by the search view):

  * old: 6.21 r/s
  * new: 5.26 r/s (15% slower)

 It makes no significant difference if I slice the new search results
 *after* the IContentListing adaptation (as I would expect, since the
 implementation in plone.app.contentlisting doesn't adapt individual items
 to IContentListingObject until they are retrieved):

  * old: 6.29 r/s
  * new: 5.27 r/s

 I'm still a bit concerned about even a 15% slowdown, given that the #1
 feature request we have on plone.uservoice.com right now is "Improve
 overall speed," and given that content listings are an important and not
 very trivially cachable part of most sites.

Ticket URL: <http://dev.plone.org/plone/ticket/9352#comment:14>
Plone <http://plone.org>
Plone Content Management System

More information about the PLIP-Advisories mailing list