You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jmeter-dev@jakarta.apache.org by se...@apache.org on 2007/09/04 17:43:54 UTC

svn commit: r572719 - /jakarta/jmeter/branches/rel-2-2/src/protocol/jdbc/org/apache/jmeter/protocol/jdbc/sampler/JDBCSampler.java

Author: sebb
Date: Tue Sep  4 08:43:54 2007
New Revision: 572719

URL: http://svn.apache.org/viewvc?rev=572719&view=rev
Log:
Add fix for Derby NPE when no ? markers provided

Modified:
    jakarta/jmeter/branches/rel-2-2/src/protocol/jdbc/org/apache/jmeter/protocol/jdbc/sampler/JDBCSampler.java

Modified: jakarta/jmeter/branches/rel-2-2/src/protocol/jdbc/org/apache/jmeter/protocol/jdbc/sampler/JDBCSampler.java
URL: http://svn.apache.org/viewvc/jakarta/jmeter/branches/rel-2-2/src/protocol/jdbc/org/apache/jmeter/protocol/jdbc/sampler/JDBCSampler.java?rev=572719&r1=572718&r2=572719&view=diff
==============================================================================
--- jakarta/jmeter/branches/rel-2-2/src/protocol/jdbc/org/apache/jmeter/protocol/jdbc/sampler/JDBCSampler.java (original)
+++ jakarta/jmeter/branches/rel-2-2/src/protocol/jdbc/org/apache/jmeter/protocol/jdbc/sampler/JDBCSampler.java Tue Sep  4 08:43:54 2007
@@ -258,7 +258,11 @@
 			String argument = arguments[i];
 			String argumentType = argumentsTypes[i];
 		    int targetSqlType = getJdbcType(argumentType);
-		    pstmt.setObject(i+1, argument, targetSqlType);
+		    try {
+				pstmt.setObject(i+1, argument, targetSqlType);
+			} catch (NullPointerException e) { // thrown by Derby JDBC (at least) if there are no "?" markers in statement
+				throw new SQLException("Could not set argument no: "+(i+1)+" - missing parameter marker?");
+			}
 		}
 	}
     



---------------------------------------------------------------------
To unsubscribe, e-mail: jmeter-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: jmeter-dev-help@jakarta.apache.org