RE: [JBoss-dev] CVS update: jbossmq/src/main/org/jboss/mq/pm/file CacheStore.java CacheStoreMBean.java

2001-11-14 Thread David Maplesden
completely, so there is no performance hit at all. Thanks David. -Original Message- From: Hiram Chirino [mailto:[EMAIL PROTECTED]] Sent: Wednesday, November 14, 2001 5:24 PM To: [EMAIL PROTECTED] Subject: [JBoss-dev] CVS update: jbossmq/src/main/org/jboss/mq/pm/file CacheStore.java

Re: [JBoss-dev] CVS update: jbossmq/src/main/org/jboss/mq/pm/file CacheStore.java CacheStoreMBean.java

2001-11-14 Thread Scott M Stark
This is the purpose of the Logger.isXXXEnabled() call. Any debugging that is executed on a per message basis should be a trace level message that is enclosed inside of a test for that priority level being active. This allows for enabling tracing with very fine control at runtime and one of the

RE: [JBoss-dev] CVS update: jbossmq/src/main/org/jboss/mq/pm/file CacheStore.java CacheStoreMBean.java

2001-11-14 Thread David Maplesden
15, 2001 9:46 AM To: [EMAIL PROTECTED] Subject: Re: [JBoss-dev] CVS update: jbossmq/src/main/org/jboss/mq/pm/file CacheStore.java CacheStoreMBean.java This is the purpose of the Logger.isXXXEnabled() call. Any debugging that is executed on a per message basis should be a trace level

RE: [JBoss-dev] CVS update: jbossmq/src/main/org/jboss/mq/pm/file CacheStore.java CacheStoreMBean.java

2001-11-14 Thread Adam Heath
On Thu, 15 Nov 2001, David Maplesden wrote: if(DEBUG) log.debug(a message); so that we can easily set the flag to false and recompile a higher performance version of the engine. With the static flag set to false the compiler removes the line completely, so there is no performance hit at

RE: [JBoss-dev] CVS update: jbossmq/src/main/org/jboss/mq/pm/file CacheStore.java CacheStoreMBean.java

2001-11-14 Thread Jason Dillon
Oh, and if we are talking about performance, I would appreciate everyone being very careful about placing debug statements inside any code that is executed for every message through the system. Several of these have been introduced with the advent of the message caching and David J's new

Re: [JBoss-dev] CVS update: jbossmq/src/main/org/jboss/mq/pm/file CacheStore.java CacheStoreMBean.java

2001-11-14 Thread Scott M Stark
- Original Message - From: David Maplesden [EMAIL PROTECTED] To: [EMAIL PROTECTED] Sent: Wednesday, November 14, 2001 12:52 PM Subject: RE: [JBoss-dev] CVS update: jbossmq/src/main/org/jboss/mq/pm/file CacheStore.java CacheStoreMBean.java Thanks, that's good to know. The amount

Re: [JBoss-dev] CVS update: jbossmq/src/main/org/jboss/mq/pm/file CacheStore.java CacheStoreMBean.java

2001-11-14 Thread David Jencks
hit at all. Thanks David. -Original Message- From: Hiram Chirino [mailto:[EMAIL PROTECTED]] Sent: Wednesday, November 14, 2001 5:24 PM To: [EMAIL PROTECTED] Subject: [JBoss-dev] CVS update: jbossmq/src/main/org/jboss/mq/pm/file CacheStore.java CacheStoreMBean.java

RE: [JBoss-dev] CVS update: jbossmq/src/main/org/jboss/mq/pm/file CacheStore.java CacheStoreMBean.java

2001-11-14 Thread David Maplesden
-Original Message- From: David Jencks [mailto:[EMAIL PROTECTED]] Sent: Thursday, November 15, 2001 10:30 AM To: [EMAIL PROTECTED] Subject: Re: [JBoss-dev] CVS update: jbossmq/src/main/org/jboss/mq/pm/file CacheStore.java CacheStoreMBean.java Oops, did you remove the excess debug

[JBoss-dev] CVS update: jbossmq/src/main/org/jboss/mq/pm/file CacheStore.java CacheStoreMBean.java

2001-11-13 Thread Hiram Chirino
User: chirino Date: 01/11/13 20:24:08 Added: src/main/org/jboss/mq/pm/file CacheStore.java CacheStoreMBean.java Log: Factored out a CacheStore object from the message store. This should lay the ground work needed so that the MessageCache can use a PM