DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT <http://nagoya.apache.org/bugzilla/show_bug.cgi?id=22831>. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND INSERTED IN THE BUG DATABASE.
http://nagoya.apache.org/bugzilla/show_bug.cgi?id=22831 when service implementation throws an exception and Log4j is used with debug enabled log file is corrupted. ------- Additional Comments From [EMAIL PROTECTED] 2003-08-30 05:11 ------- comments to PATCH: AxisFault and AxisServlet classes are modified to get rid of recursive calls, rearranged the order of statements and one expression cached , probably modification is needed in classes other than AxisServlet when transport is other than http like in org.apache.axis.transport.mail.MailWorker class. tests run successfully. originally i had NDC in the code to see what messages were actually generated during AxisFault logging and not to confuse them with other mesages. when i run ant target all-tests i got a message test failed - dependency on log4j. there were discussions on commons logging mailing lists to wrap NDC in future, no dates so. it looks that in this particular case NDC is very usefull. of course it introduces a compile time dependency on log4j, but should not be any runtime dependency unless you actually use log4j (it was all wrapped in if statements). Axis shipped with log4j so compile time dependency should not be an issue. the pach is without NDC. if compile time dependency is allowed but not runtime i will be more than happy to submit a patch with NDC. don't you miss conditional compilation :) oleg
