Re: [Zope-CMF] [CMF 2.1] opaque items calls make performance issue

2009-08-11 Thread yuppie
Hi! Hanno Schlichting wrote: I don't know how such a feature removal or deprecation would be handled on the CMF level. AFAICS IWorkflowAware and IOpaqueItemManager methods (as defined in CMF 2.2) were added to CMFCatalogAware because at that time events were not available and it was easier

Re: [Zope-CMF] [CMF 2.1] opaque items calls make performance issue

2009-08-11 Thread Charlie Clark
Am 11.08.2009, 20:06 Uhr, schrieb yuppie y.2...@wcm-solutions.de: I propose to split up CMFCatalogAware in CatalogAware, WorkflowAware and OpaqueItemManager mixins. All the interfaces provided by these base classes should be optional for CMF content. PortalFolderBase e.g. is not catalog or

Re: [Zope-CMF] [CMF 2.1] opaque items calls make performance issue

2009-08-10 Thread toutpt
Charlie Clark a écrit : Am 09.08.2009, 04:48 Uhr, schrieb Martin Aspeli optilude+li...@gmail.com: self_base = aq_base(self) for name in self_base.__dict__.keys() obj = getattr(self, name) if ICallableOpaqueItem.providedBy(obj): items.append((obj.getId(), obj))

Re: [Zope-CMF] [CMF 2.1] opaque items calls make performance issue

2009-08-10 Thread Jens Vagelpohl
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On Aug 9, 2009, at 21:03 , Charlie Clark wrote: From my previous answer to this which I forgot to send to the list as well - can we change the list settings so that answers go to it by default? No. See

Re: [Zope-CMF] [CMF 2.1] opaque items calls make performance issue

2009-08-10 Thread toutpt
Charlie Clark a écrit : Am 10.08.2009, 08:45 Uhr, schrieb toutpt tou...@gmail.com: Thank you Charlie for this answer and this Read The Fucking Interfaces . Well, although it's a truism that if you read the code you'll find the answer, it does also take a while to get used to

Re: [Zope-CMF] [CMF 2.1] opaque items calls make performance issue

2009-08-09 Thread Charlie Clark
Am 09.08.2009, 04:48 Uhr, schrieb Martin Aspeli optilude+li...@gmail.com: self_base = aq_base(self) for name in self_base.__dict__.keys() obj = getattr(self, name) if ICallableOpaqueItem.providedBy(obj): items.append((obj.getId(), obj)) That's just *nuts*! From my previous

Re: [Zope-CMF] [CMF 2.1] opaque items calls make performance issue

2009-08-08 Thread Hanno Schlichting
On Sat, Aug 8, 2009 at 12:14 AM, Charlie Clarkchar...@begeistert.org wrote: Is the patch for Plone or CMF? It's for Plone, I guess. The problem is solved in Plone 4.0 itself (via monkey-patching CMF). If it's for CMF then you should consider simply submitting a patch or opening a branch. But

Re: [Zope-CMF] [CMF 2.1] opaque items calls make performance issue

2009-08-08 Thread Wichert Akkerman
On 2009-8-8 11:53, Hanno Schlichting wrote: On Sat, Aug 8, 2009 at 12:14 AM, Charlie Clarkchar...@begeistert.org wrote: Is the patch for Plone or CMF? It's for Plone, I guess. The problem is solved in Plone 4.0 itself (via monkey-patching CMF). If it's for CMF then you should consider

Re: [Zope-CMF] [CMF 2.1] opaque items calls make performance issue

2009-08-08 Thread Martin Aspeli
Wichert Akkerman wrote: On 2009-8-8 11:53, Hanno Schlichting wrote: On Sat, Aug 8, 2009 at 12:14 AM, Charlie Clarkchar...@begeistert.org wrote: Is the patch for Plone or CMF? It's for Plone, I guess. The problem is solved in Plone 4.0 itself (via monkey-patching CMF). If it's for CMF

Re: [Zope-CMF] [CMF 2.1] opaque items calls make performance issue

2009-08-07 Thread Charlie Clark
Am 07.08.2009, 22:17 Uhr, schrieb toutpt tou...@gmail.com: experimental.opaquespeedup is good but not enough for me. The load tests results are not enough, I want to be more 'aggressive' and experimental. For me this package is backward compatible and do a catalog query instead of parsing

Re: [Zope-CMF] [CMF 2.1] opaque items calls make performance issue

2009-08-05 Thread toutpt
Hanno Schlichting a écrit : On Mon, Aug 3, 2009 at 11:09 PM, toutpttou...@gmail.com wrote: I don't really understand why all these improvements are not merged inside plone3.X and CMF and are marked up as 'experimental'. The opaque items issue at hand is not a simple and clear bug

[Zope-CMF] [CMF 2.1] opaque items calls make performance issue

2009-08-03 Thread toutpt
Hi, I m currently profiling the code of Plone to find performance issue and fix them (i m sure there are a lot). There are things I don't understand very well, so sorry if I m wrong. The code of CMFCatalogAware.opaqueItems parse every attributes of plonesite and evaluate this for each

Re: [Zope-CMF] [CMF 2.1] opaque items calls make performance issue

2009-08-03 Thread Hanno Schlichting
On Mon, Aug 3, 2009 at 10:46 PM, toutpttou...@gmail.com wrote: I m currently profiling the code of Plone to find performance issue and fix them (i m sure there are a lot). Did you try to install all the experimental.* (opaquespeedup, contentcreation, catalogqueryplan) packages that have been

Re: [Zope-CMF] [CMF 2.1] opaque items calls make performance issue

2009-08-03 Thread toutpt
Hanno Schlichting a écrit : On Mon, Aug 3, 2009 at 10:46 PM, toutpttou...@gmail.com wrote: I m currently profiling the code of Plone to find performance issue and fix them (i m sure there are a lot). Did you try to install all the experimental.* (opaquespeedup, contentcreation,

Re: [Zope-CMF] [CMF 2.1] opaque items calls make performance issue

2009-08-03 Thread Hanno Schlichting
On Mon, Aug 3, 2009 at 11:09 PM, toutpttou...@gmail.com wrote: I don't really understand why all these improvements are not merged inside plone3.X and CMF and are marked up as 'experimental'. The opaque items issue at hand is not a simple and clear bug fix. On the Plone level we have decided to