You are viewing a plain text version of this content. The canonical link for it is here.

Posted to java-dev@axis.apache.org by bu...@apache.org on 2002/11/15 19:22:21 UTC

DO NOT REPLY [Bug 14602] New: - SOAPPart.setCurrentForm serializes complete message when debug is not enabled

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: axis-dev@xml.apache.org
        ReportedBy: Eric.D.Friedman@wellsfargo.com


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.