Bill Barker wrote:
> <ma...@apache.org> wrote in message 
> news:20091201184432.f01322388...@eris.apache.org...
>> Author: markt
>> Date: Tue Dec  1 18:44:32 2009
>> New Revision: 885860
>>
>> URL: http://svn.apache.org/viewvc?rev=885860&view=rev
>> Log:
>> More memory leak protection - this time for the GC Daemon thread.
>>
>> log.error(sm.getString("jreLeakListener.gcDaemonFail"), e);
> 
> This should be no higher than INFO (and, personally, would go for DEBUG). 
> It just means that you are not running a Sun JVM.  IMHO, having this one at 
> ERROR just produces messages on the user list of the form "I got this big 
> scary message in my log file while running Tomcat on the latest IBM JVM. 
> What do I do about it?".

Fair point. I thought I had caught all of those after my original
copy/paste of the error handling. Clearly not.

I'll get that fixed and probably add a test to see if we are on a Sun
JVM and log accordingly.

Cheers,

Mark




---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to