The AXIS community had decided that what I was doing was overkill.  NEW
debug messages weren't going to be NLS'ed.  No-one volunteered to clean up
what had been done, though.  Looks like Richard's doing it.

Russell Butek
[EMAIL PROTECTED]


Tom Jordahl <[EMAIL PROTECTED]> on 05/09/2002 01:18:55 PM

Please respond to [EMAIL PROTECTED]

To:    "'[EMAIL PROTECTED]'" <[EMAIL PROTECTED]>
cc:
Subject:    RE: cvs commit: xml-axis/java/src/org/apache/axis
       FaultableHandle  r.java



> Removed NLS from enter/exit log.debug

While I am +1 to this, this isn't what has been done in the past.

Russell has argued that IBM puts all messages, even debug, in to the
message
file and that Axis should too.

--
Tom Jordahl


-----Original Message-----
From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED]]
Sent: Thursday, May 09, 2002 2:15 PM
To: [EMAIL PROTECTED]
Subject: cvs commit: xml-axis/java/src/org/apache/axis
FaultableHandler.java


rsitze      02/05/09 11:14:49

  Modified:    java/src/org/apache/axis FaultableHandler.java
  Log:
  Removed NLS from enter/exit log.debug
  Logged wrapped exception as info instead of error, as per guidelines.

  Revision  Changes    Path
  1.34      +5 -5
  xml-axis/java/src/org/apache/axis/FaultableHandler.java

  Index: FaultableHandler.java
  ===================================================================
  RCS file:
  /home/cvs/xml-axis/java/src/org/apache/axis/FaultableHandler.java,v
  retrieving revision 1.33
  retrieving revision 1.34
  diff -u -r1.33 -r1.34
  --- FaultableHandler.java   25 Feb 2002 17:38:14 -0000    1.33
  +++ FaultableHandler.java   9 May 2002 18:14:49 -0000     1.34
  @@ -103,12 +103,12 @@
        * and already done its fault processing - as needed.
        */
       public void invoke(MessageContext msgContext) throws AxisFault {
  -        log.debug(JavaUtils.getMessage("enter00",
  "FaultableHandler::invoke"));
  +        log.debug("Enter: FaultableHandler::invoke");
           try {
               workHandler.invoke( msgContext );
           }
           catch( Exception e ) {
  -            log.error( JavaUtils.getMessage("exception00"), e );
  +            log.info(JavaUtils.getMessage("toAxisFault00"), e );
               AxisFault fault = AxisFault.makeFault(e);

               AxisEngine engine = msgContext.getAxisEngine();
  @@ -146,16 +146,16 @@
                   throw fault;
               }
           }
  -        log.debug(JavaUtils.getMessage("exit00",
  "FaultableHandler::invoke"));
  +        log.debug("Exit: FaultableHandler::invoke");
       }

       /**
        * Some handler later on has faulted so we need to process the
        fault.
        */
       public void onFault(MessageContext msgContext) {
  -        log.debug(JavaUtils.getMessage("enter00",
  "FaultableHandler::onFault"));
  +        log.debug("Enter: FaultableHandler::onFault");
           workHandler.onFault( msgContext );
  -        log.debug(JavaUtils.getMessage("exit00",
  "FaultableHandler::onFault"));
  +        log.debug("Exit: FaultableHandler::onFault");
       };

       public boolean canHandleBlock(QName qname) {





Reply via email to