Changeset: 8c4900034b84 for MonetDB
URL: http://dev.monetdb.org/hg/MonetDB?cmd=changeset;node=8c4900034b84
Modified Files:
        java/src/nl/cwi/monetdb/jdbc/MonetDatabaseMetaData.java
Branch: Oct2010
Log Message:

Fix bug #2703, don't try to be clever, just use a new Statement for each query 
such that multiple metadata ResultSets can be open at the same time


diffs (27 lines):

diff -r 08d785b8f427 -r 8c4900034b84 
java/src/nl/cwi/monetdb/jdbc/MonetDatabaseMetaData.java
--- a/java/src/nl/cwi/monetdb/jdbc/MonetDatabaseMetaData.java   Fri Oct 22 
10:58:44 2010 +0200
+++ b/java/src/nl/cwi/monetdb/jdbc/MonetDatabaseMetaData.java   Fri Oct 22 
14:51:54 2010 +0200
@@ -32,7 +32,6 @@
 public class MonetDatabaseMetaData implements DatabaseMetaData {
        private Connection con;
        private Driver driver;
-       private Statement stmt;
        private static Map envs = new HashMap();
 
        public MonetDatabaseMetaData(Connection parent) {
@@ -42,10 +41,11 @@
 
        private synchronized Statement getStmt() throws SQLException {
                // use Statement which allows scrolling both directions through 
results
-               if (stmt == null)
-                       stmt = 
con.createStatement(ResultSet.TYPE_SCROLL_INSENSITIVE, 
ResultSet.CONCUR_READ_ONLY);
-
-               return(stmt);
+               // cannot reuse stmt here, as people may request multiple
+               // queries, see for example bug #2703
+               return(con.createStatement(
+                                       ResultSet.TYPE_SCROLL_INSENSITIVE,
+                                       ResultSet.CONCUR_READ_ONLY));
        }
 
        /**
_______________________________________________
Checkin-list mailing list
[email protected]
http://mail.monetdb.org/mailman/listinfo/checkin-list

Reply via email to