You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cocoon.apache.org by Vadim Gritsenko <vg...@hns.com> on 2001/06/07 21:55:20 UTC

[Patch] Exception handling in Caching*Pipeline and Abstract*Pipeline

Hi everybody,

I found that mentioned pipelines catch all exceptions in their process() method
and wrap them into ProcessingException. It done this way for all exceptions,
inluding ProcessingException!

I think that this is quite excessive and expensive too: in my setup every
ResourceNotFound exception ends up with around 10K in the log file.

Can somebody apply this simple patch? It just re-trows ProcessingException as-is,
without creating new exception instance.

Thanks,

Vadim

RE: [Patch] Exception handling in Caching*Pipeline and Abstract*Pipeline

Posted by Vadim Gritsenko <vg...@hns.com>.
Looks great.

Thanks,
Vadim

> -----Original Message-----
> From: giacomo [mailto:giacomo@apache.org]
> Sent: Thursday, June 07, 2001 16:34
> To: Cocoon Developers
> Subject: Re: [Patch] Exception handling in Caching*Pipeline and Abstract*Pipeline
> 
> 
> On Thu, 7 Jun 2001, Vadim Gritsenko wrote:
> 
> Thanks, Vadim. Patch applied, please cross check.
> 
> Giacomo
> 
> > Hi everybody,
> >
> > I found that mentioned pipelines catch all exceptions in their process() method
> > and wrap them into ProcessingException. It done this way for all exceptions,
> > inluding ProcessingException!
> >
> > I think that this is quite excessive and expensive too: in my setup every
> > ResourceNotFound exception ends up with around 10K in the log file.
> >
> > Can somebody apply this simple patch? It just re-trows ProcessingException as-is,
> > without creating new exception instance.
> >
> > Thanks,
> >
> > Vadim
> 
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: cocoon-dev-unsubscribe@xml.apache.org
> For additional commands, email: cocoon-dev-help@xml.apache.org
> 
> 

---------------------------------------------------------------------
To unsubscribe, e-mail: cocoon-dev-unsubscribe@xml.apache.org
For additional commands, email: cocoon-dev-help@xml.apache.org

Re: [Patch] Exception handling in Caching*Pipeline and Abstract*Pipeline

Posted by giacomo <gi...@apache.org>.
On Thu, 7 Jun 2001, Vadim Gritsenko wrote:

Thanks, Vadim. Patch applied, please cross check.

Giacomo

> Hi everybody,
>
> I found that mentioned pipelines catch all exceptions in their process() method
> and wrap them into ProcessingException. It done this way for all exceptions,
> inluding ProcessingException!
>
> I think that this is quite excessive and expensive too: in my setup every
> ResourceNotFound exception ends up with around 10K in the log file.
>
> Can somebody apply this simple patch? It just re-trows ProcessingException as-is,
> without creating new exception instance.
>
> Thanks,
>
> Vadim


---------------------------------------------------------------------
To unsubscribe, e-mail: cocoon-dev-unsubscribe@xml.apache.org
For additional commands, email: cocoon-dev-help@xml.apache.org