Bug 50173

Summary: JDBCSampler discards ResultSet from a PreparedStatement.
Product: JMeter - Now in Github Reporter: whitingjr <jeremy.whiting>
Component: MainAssignee: JMeter issues mailing list <issues>
Status: RESOLVED FIXED    
Severity: normal    
Priority: P2    
Version: 2.4   
Target Milestone: ---   
Hardware: All   
OS: All   
Attachments: File with SVN diff patch details.

Description whitingjr 2010-10-28 10:14:39 UTC
Hi,
 I am using JMeter with a Test Plan that uses a JDBC Request with a PreparedStatement query type.
 When running this I am getting the following error in the jmeter.log file

2010/10/28 11:30:31 INFO  - jmeter.threads.JMeterThread: Thread started: JDBC Users 1-1 
2010/10/28 12:13:38 ERROR - jmeter.threads.JMeterThread: Error while processing sampler 'PREPARED SELECT Customer JDBC Request' : java.lang.NullPointerException
	at org.apache.jmeter.protocol.jdbc.sampler.JDBCSampler.getStringFromResultSet(JDBCSampler.java:415)
	at org.apache.jmeter.protocol.jdbc.sampler.JDBCSampler.resultSetsToString(JDBCSampler.java:268)
	at org.apache.jmeter.protocol.jdbc.sampler.JDBCSampler.sample(JDBCSampler.java:208)
	at org.apache.jmeter.threads.JMeterThread.process_sampler(JMeterThread.java:348)
	at org.apache.jmeter.threads.JMeterThread.run(JMeterThread.java:243)
	at java.lang.Thread.run(Thread.java:619)

 The query (below) can be expected to return a number of records 
SELECT * FROM Customer WHERE ID=?;
the param is set to 1

 If I use instead a SelectStatement results are returned so the query itself is not at fault.

 I have retrieved the code for the v2_4 tag to identify the root cause for the NPE and create a patch. This bug report includes a patch against trunk as the issue is also in trunk. 
 The root cause is due to:
org.apache.jmeter.protocol.jdbc.sampler.JDBCSampler.sample(Entry e)
 
 The code path for PreparedStatements types starting on line 204 has in it a call to
org.hsqldb.jdbc.JDBCPreparedStatement.executeUpdate() 
which returns a ResultSet. The side effect of the call is to set the JDBCPreparedStatement.currentResultSet field member to null.
 JDBCSampler does not keep the ResultSet reference. Instead it is discarded.

 The call on the next line (208)
String sb = resultSetsToString(pstmt,true,null)
 again tries to get the ResultSet on line 267. This causes a NullPointerException because the JDBCPreparedStatement.currentResultSet field was set to null earlier.

 Looking at the code path for a CALLABLE statement type (line 190) shows better handling for queries that return ResultSet.

 The two lines 
207,208

 replaced with
                boolean hasResultSet = pstmt.execute();
                String sb = resultSetsToString(pstmt,hasResultSet,null);
                
 fixes the defect.

Regards,
Jeremy Whiting
Red Hat
Comment 1 whitingjr 2010-10-28 10:18:29 UTC
Created attachment 26222 [details]
File with SVN diff patch details.

Attempted to run the Ant test target but this failed.

BUILD FAILED
JMeter-trunk/build.xml:1723: <fixcrlf> error: srcdir does not exist: 'JMeter-trunk/bin/testfiles'
Comment 2 Sebb 2010-10-28 18:07:31 UTC
(In reply to comment #1)
> Attempted to run the Ant test target but this failed.
> 
> BUILD FAILED
> JMeter-trunk/build.xml:1723: <fixcrlf> error: srcdir does not exist:
> 'JMeter-trunk/bin/testfiles'

In that case, you have not downloaded the full source.
Comment 3 Sebb 2010-10-28 18:14:00 UTC
Thanks for the patch, applied to SVN:

URL: http://svn.apache.org/viewvc?rev=1028518&view=rev
Log:
Bug 50173 - JDBCSampler discards ResultSet from a PreparedStatement

Modified:
   jakarta/jmeter/trunk/src/protocol/jdbc/org/apache/jmeter/protocol/jdbc/sampler/JDBCSampler.java
   jakarta/jmeter/trunk/xdocs/changes.xml
Comment 4 The ASF infrastructure team 2022-09-24 20:37:45 UTC
This issue has been migrated to GitHub: https://github.com/apache/jmeter/issues/2418