Re: svn commit: r161067 - cocoon/branches/BRANCH_2_1_X/src/java/org/apache/cocoon/components/pipeline/AbstractProcessingPipeline.java

[Vadim Gritsenko]

[EMAIL PROTECTED] 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