From: Greg Sabino Mullane <greg@endpoint.com>
Date: Wed, 9 Feb 2011 14:36:33 +0000 (-0500)
Subject: Exclude idle in transaction from the query_time per bug #43.
X-Git-Tag: 2.18.0~91
X-Git-Url: http://waps.l3s.uni-hannover.de/gitweb/?a=commitdiff_plain;h=ffb00529799a8dbb4e9f82e0d6c94b523e02161d;p=check_postgres.git

Exclude idle in transaction from the query_time per bug #43.
Since we've just overhauled this check, seems like a good a time
as any to make this change.
---

diff --git a/check_postgres.pl b/check_postgres.pl
index 96314628b..7b5bb11dc 100755
--- a/check_postgres.pl
+++ b/check_postgres.pl
@@ -4922,7 +4922,7 @@ sub check_query_time {
                    msg('queries'),
                    msg('query-time'),
                    'query_start',
-                   q{query_start IS NOT NULL});
+                   q{query_start IS NOT NULL AND current_query <> '<IDLE> in transaction'});
 
     return;
 
@@ -8441,9 +8441,9 @@ line lists the database.
 
 =head2 B<query_time>
 
-(C<symlink: check_postgres_query_time>) Checks the length of running queries on one or more databases. There is 
-no need to run this more than once on the same database cluster.
-Databases can be filtered 
+(C<symlink: check_postgres_query_time>) Checks the length of running queries on one or more databases. 
+There is no need to run this more than once on the same database cluster. Note that 
+this already excludes queries that are "idle in transaction". Databases can be filtered 
 by using the I<--include> and I<--exclude> options. See the L</"BASIC FILTERING">
 section for more details. You can also filter on the user running the 
 query with the I<--includeuser> and I<--excludeuser> options.
@@ -9018,6 +9018,8 @@ Items not specifically attributed are by Greg Sabino Mullane.
 
   Standardize and clean up all perfdata ouput (bug #52)
 
+  Exclude "idle in transaction" from the query_time check (bug #43)
+
   Clean up the custom_query action a bit.
 
 =item B<Version 2.16.0> January 20, 2011