From 2714cb1f7e4a19bf24dbd981443bde1b425e4109 Mon Sep 17 00:00:00 2001 From: Josh Schlesinger Date: Wed, 10 Sep 2014 13:19:49 -0500 Subject: [PATCH] using rs.next() instead of rs.first() to get the first record to work around an issue in the sql server jdbc driver --- src/main/java/com/appdynamics/monitors/sql/SQLMonitor.java | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/main/java/com/appdynamics/monitors/sql/SQLMonitor.java b/src/main/java/com/appdynamics/monitors/sql/SQLMonitor.java index 1d4acbd..e9043d9 100644 --- a/src/main/java/com/appdynamics/monitors/sql/SQLMonitor.java +++ b/src/main/java/com/appdynamics/monitors/sql/SQLMonitor.java @@ -151,7 +151,11 @@ private Data executeQuery(Connection conn, String query) { stmt = conn.createStatement(); rs = stmt.executeQuery(query); // only get the first result - rs.first(); + if(!rs.next()) { + logger.info("0 results returned by query"); + return retval; + } + // only get the first column String value = rs.getString(1); // use the lable for the name of the metric