You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by kf...@apache.org on 2015/10/29 09:33:15 UTC
svn commit: r1711191 - in /tomcat/tc8.0.x/trunk:
modules/jdbc-pool/doc/jdbc-pool.xml
modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/interceptor/SlowQueryReport.java
webapps/docs/changelog.xml
Author: kfujino
Date: Thu Oct 29 08:33:15 2015
New Revision: 1711191
URL: http://svn.apache.org/viewvc?rev=1711191&view=rev
Log:
When creating a QueryStats object, ensure that maxQueries is checked.
If maxQueries is a value less than or equal to 0, QueryStats are never created.
Modified:
tomcat/tc8.0.x/trunk/modules/jdbc-pool/doc/jdbc-pool.xml
tomcat/tc8.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/interceptor/SlowQueryReport.java
tomcat/tc8.0.x/trunk/webapps/docs/changelog.xml
Modified: tomcat/tc8.0.x/trunk/modules/jdbc-pool/doc/jdbc-pool.xml
URL: http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/modules/jdbc-pool/doc/jdbc-pool.xml?rev=1711191&r1=1711190&r2=1711191&view=diff
==============================================================================
--- tomcat/tc8.0.x/trunk/modules/jdbc-pool/doc/jdbc-pool.xml (original)
+++ tomcat/tc8.0.x/trunk/modules/jdbc-pool/doc/jdbc-pool.xml Thu Oct 29 08:33:15 2015
@@ -694,6 +694,7 @@
</attribute>
<attribute name="maxQueries" required="false">
<p>(int as String) The maximum number of queries to keep track of in order to preserve memory space.
+ A value less than or equal to 0 will disable this feature.
The default value is <code>1000</code>.
</p>
</attribute>
Modified: tomcat/tc8.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/interceptor/SlowQueryReport.java
URL: http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/interceptor/SlowQueryReport.java?rev=1711191&r1=1711190&r2=1711191&view=diff
==============================================================================
--- tomcat/tc8.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/interceptor/SlowQueryReport.java (original)
+++ tomcat/tc8.0.x/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/interceptor/SlowQueryReport.java Thu Oct 29 08:33:15 2015
@@ -148,14 +148,18 @@ public class SlowQueryReport extends Abs
@Override
public void prepareStatement(String sql, long time) {
- QueryStats qs = getQueryStats(sql);
- if (qs != null) qs.prepare(time);
+ if (this.maxQueries > 0 ) {
+ QueryStats qs = getQueryStats(sql);
+ if (qs != null) qs.prepare(time);
+ }
}
@Override
public void prepareCall(String sql, long time) {
- QueryStats qs = getQueryStats(sql);
- if (qs != null) qs.prepare(time);
+ if (this.maxQueries > 0 ) {
+ QueryStats qs = getQueryStats(sql);
+ if (qs != null) qs.prepare(time);
+ }
}
/**
Modified: tomcat/tc8.0.x/trunk/webapps/docs/changelog.xml
URL: http://svn.apache.org/viewvc/tomcat/tc8.0.x/trunk/webapps/docs/changelog.xml?rev=1711191&r1=1711190&r2=1711191&view=diff
==============================================================================
--- tomcat/tc8.0.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc8.0.x/trunk/webapps/docs/changelog.xml Thu Oct 29 08:33:15 2015
@@ -152,6 +152,12 @@
<bug>58489</bug>: Correct QueryStatsComparator to hold up the
general contract for Comparator. (fschumacher)
</fix>
+ <fix>
+ When creating a <code>QueryStats</code> object, ensure that
+ <code>maxQueries</code> is checked. If <code>maxQueries</code> is a
+ value less than or equal to 0, <code>QueryStats</code> are never
+ created. (kfujino)
+ </fix>
</changelog>
</subsection>
<subsection name="Other">
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org