cocoon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Vadim Gritsenko <va...@reverycodes.com>
Subject Re: svn commit: r161067 - cocoon/branches/BRANCH_2_1_X/src/java/org/apache/cocoon/components/pipeline/AbstractProcessingPipeline.java
Date Tue, 12 Apr 2005 16:50:02 GMT
sylvain@apache.org wrote:
> 
>              } catch (ProcessingException e) {
> +                // Log the original exception
> +                getLogger().error("Failed to process error handler for exception", e);
>                  throw e;

I think we had an agreement against mile-long log files. Why log exception here 
if it will be logged later on (by the caller)?

>              } catch (Exception e) {
>                  throw new ProcessingException("Failed to handle exception <" + ex
+ ">", e);
>              }

(note: here it is logged at all)

Vadim

Mime
View raw message