If I have time over the next day or so, I'm going to go through the most accessed jetspeed code, and put in the debug tests around log.debug calls. I'll also see if there are some .info's that should be downgraded to debug, and visa versa.
For example, the castor psml manager info's what it's doing, and these should probably be debugs. Anybody worried? - Glenn -------------------------------------------- Glenn R. Golden, Systems Research Programmer University of Michigan School of Information [EMAIL PROTECTED] 734-615-1419 -------------------------------------------- > -----Original Message----- > From: David Sean Taylor [mailto:[EMAIL PROTECTED]] > Sent: Monday, July 01, 2002 3:51 PM > To: 'Jetspeed Developers List' > Subject: RE: Opinion on log messages > > > > > > -----Original Message----- > > From: Glenn Golden [mailto:[EMAIL PROTECTED]] > > Sent: Monday, July 01, 2002 12:44 PM > > To: 'Jetspeed-Dev' > > Subject: Opinion on log messages > > > > > > I'm looking at JetspeedRunDataService, at the log messages. > > There's on in each of the three routines. These end up in > > the jetspeed log quite a bit. > > > > I was thinking about removing them. > > > > But looking at them, they are good "debug" messages, even > > perhaps good "info" messages - to give us a feel from looking > > at the logs of what's going on. They are also properly > > guarded with an "if debug logging" test. > > +1 for guarding all debug messages with "if debug logging" > Otherwise potentially complex statements will be evaluated, > objects created etc., for no reason at all Jetspeed hasn't > been to good about this in the past > > > > > So, I think I'll leave them, and if I don't want them, I turn > > off debug logging. > > > > I just want to check with the other developers to see if this > > logic sits well with you. > > > > Thanks. > > > > - Glenn > > > > -------------------------------------------- > > Glenn R. Golden, Systems Research Programmer > > University of Michigan School of Information > > [EMAIL PROTECTED] 734-615-1419 > > -------------------------------------------- > > > > -- > > To unsubscribe, e-mail: > > <mailto:jetspeed-dev-> [EMAIL PROTECTED]> > > For > > additional commands, > > e-mail: <mailto:[EMAIL PROTECTED]> > > > > > > > > -- > To unsubscribe, e-mail: > <mailto:jetspeed-dev-> [EMAIL PROTECTED]> > For > additional commands, > e-mail: <mailto:[EMAIL PROTECTED]> > -- To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
