While there are many places in the system where people integrating Axis will want to 
be using logging, there are also many places (esp the samples) where there are end 
users directly using Axis clients.  I agree with Doug that it's kind of silly for 
those people to have to change their log4j configuration to get at errors which make 
sense to be sent to the console.  If that's what was happening here, it's a dandy fix. 
 Doug, I think, was claiming that there are now too many places where we rely on log4j 
for output where System.err actually would be appropriate.

(this particular situation should have been ERROR output anyway, not DEBUG - perhaps 
we could add an argument to AxisFault.dump() to take a priority, or split it into 
logDebug()/logError())

--Glen

> -----Original Message-----
> From: Richard Sitze [mailto:[EMAIL PROTECTED]]
> Sent: Tuesday, March 12, 2002 4:25 PM
> To: [EMAIL PROTECTED]
> Subject: Recent change to stock/GetQuote.java
> 
> 
> I can understand your desire to see this for your immediate 
> purposes, but
> this solution is counter productive when you integrate AXIS into a
> production environment or product.
> 
> If you prefer, I would be happy to enable DEBUG output to an 
> "axis.log"
> file, but not to the screen (Rich, who sits some 3 feet 
> behind me, will
> likely whack the back of my head if I start dumping such lines to the
> console).
> 
> I think the pattern that seems most innocuous is to be notified of the
> error on the console, and dig through the log for the details.  If the
> community has another opinion then I think it should be debated as a
> "design point", rather than tweeking individual lines to help.
> 
> <ras>
> 
> *******************************************
> Richard A. Sitze            [EMAIL PROTECTED]
> CORBA Interoperability & WebServices
> IBM WebSphere Development
> 
> 
>                                                               
>                                           
>                       [EMAIL PROTECTED]                          
>                                           
>                                                To:      
> [EMAIL PROTECTED]                         
>                       03/12/2002 03:00         cc:            
>                                           
>                       PM                       Subject: cvs 
> commit: xml-axis/java/samples/stock         
>                       Please respond           GetQuote.java  
>                                           
>                       to axis-dev                             
>                                           
>                                                               
>                                           
>                                                               
>                                           
> 
> 
> 
> 
> dug         02/03/12 13:00:34
> 
>   Modified:    java/samples/stock GetQuote.java
>   Log:
>   Make error message go to the screen instead of some mystery log file
>   that newbies will never find.
> 
>   Revision  Changes    Path
>   1.38      +1 -1      xml-axis/java/samples/stock/GetQuote.java
> 
>   Index: GetQuote.java
>   ===================================================================
>   RCS file: /home/cvs/xml-axis/java/samples/stock/GetQuote.java,v
>   retrieving revision 1.37
>   retrieving revision 1.38
>   diff -u -r1.37 -r1.38
>   --- GetQuote.java            8 Feb 2002 22:14:11 -0000      
>      1.37
>   +++ GetQuote.java            12 Mar 2002 21:00:34 -0000     
>      1.38
>   @@ -138,7 +138,7 @@
>        }
>        catch( Exception e ) {
>            if ( e instanceof AxisFault ) {
>   -            ((AxisFault)e).dump();
>   +            System.err.println( ((AxisFault)e).dumpToString() );
>            } else
>                e.printStackTrace();
>        }
> 
> 
> 
> 
> 
> 

Reply via email to