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=14602>.
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=14602

SOAPPart.setCurrentForm serializes complete message when debug is not enabled

           Summary: SOAPPart.setCurrentForm serializes complete message when
                    debug is not enabled
           Product: Axis
           Version: 1.0
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: Normal
          Priority: Other
         Component: Basic Architecture
        AssignedTo: [EMAIL PROTECTED]
        ReportedBy: [EMAIL PROTECTED]


the setCurrentForm method on SOAPPart has as its first line a log.debug()
message in which the message is concatenated as a string with a log message. 
That triggers serialization of the message, which is quite expensive.  

This log.debug() should, IMHO, be wrapped in a test to determine if debug
logging is enabled, so that it doesn't happen unless it needs to.

Reply via email to