Don't return -1 when performing an update and a result set is returned.

JumpMind · Mar 17, 2013 · 3c881f4 · 3c881f4
1 parent 4e4ab0f
commit 3c881f4
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/symmetric-jdbc/src/main/java/org/jumpmind/db/sql/JdbcSqlTemplate.java b/symmetric-jdbc/src/main/java/org/jumpmind/db/sql/JdbcSqlTemplate.java
@@ -286,7 +286,8 @@ public Integer execute(Connection con) throws SQLException {
                         stmt = con.createStatement();
                         stmt.setQueryTimeout(settings.getQueryTimeout());
                         stmt.execute(sql);
-                        return stmt.getUpdateCount();
+                        int updateCount = stmt.getUpdateCount();
+                        return updateCount > 0 ? updateCount : 0;
                     } finally {
                         close(stmt);
                     }
@@ -302,7 +303,8 @@ public Integer execute(Connection con) throws SQLException {
                             setValues(ps, args);
                         }
                         ps.execute();
-                        return ps.getUpdateCount();
+                        int updateCount = ps.getUpdateCount();
+                        return updateCount > 0 ? updateCount : 0;
                     } finally {
                         close(ps);
                     }
@@ -338,6 +340,7 @@ public Integer execute(Connection con) throws SQLException {
                         try {
                             boolean hasResults = stmt.execute(statement);
                             int updateCount = stmt.getUpdateCount();
+                            updateCount = updateCount > 0 ? updateCount : 0;
                             totalUpdateCount += updateCount;
                             int rowsRetrieved = 0;
                             if (hasResults) {