Mailing-List: contact dev-help@cocoon.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cocoon.apache.org
Message-ID: <35647.10.0.0.5.1067780548.squirrel@ags01.agsoftware.dnsalias.com>
Date: Sun, 2 Nov 2003 07:42:28 -0600 (CST)
Subject: Re: Profiling Pipeline [was Re: [RT] Unit testing and CocoonUnit]
From: "Antonio Gallardo" <agallardo@agsoftware.dnsalias.com>
To: <dev@cocoon.apache.org>
In-Reply-To: <F99B2F6D-0D17-11D8-8009-000393D2CB02@apache.org>
References: <3FA433DD.8050102@mm.st>
        <F99B2F6D-0D17-11D8-8009-000393D2CB02@apache.org>
Importance: Normal
MIME-Version: 1.0
Content-Type: text/plain; charset=iso-8859-1
Content-Transfer-Encoding: 8bit

Stefano Mazzocchi dijo:
>
> On Saturday, Nov 1, 2003, at 23:29 Europe/Rome, Steve K wrote:
>
>> And finally, on a somewhat unrelated subject, one thing that I've
>> always wanted Cocoon to do may be possible if support for collecting
>> the XML at each pipeline step is added.  To aid in debugging, I think
>> it would be very helpful to switch on some kind of debug mode, that
>> would cause a trace of what pipeline steps where executed and the
>> state of the XML at each step to be printed out at the bottom of each
>> page you output to the browser.  This way it is easy for a developer
>> to see the path though the pipelines the request took, as well as a
>> snapshot of the XML each step of the way.
>
> This is already there, althought somewhat hidden, check into the
> "profiler" block.
>
> BTW, there is something that always bugged me about the profiler: the
> time that gives you is almost totally useless, while the exposed view
> of the pipeline internals is a *great* debugging tool (some people do
> it with views, but sometimes you don't know where the problem is so you
> might want to see it all).
>
> I propose two changes here:
>
> 1) rename the "profiling" pipeline into "debug"
> 2) remove the timings (they don't make any sense)
> 3) move the whole thing into core

> WTYT?

+1 on 1)

+- ? 2) What are the timings? Sorry, I don't understand this point.

-1 on 3) It is desired to have a very light core. If the core can live
without the profiler please don't include them on every distribution. I
mean development and production.

Best Regards,

Antonio Gallardo