[DM-41951] Set auto commit to false

This seems to be holding a lot of memory up on the server by holding
open a transaction, but we don't need a transaction just for SELECTs.

src/main/java/org/opencadc/tap/impl/QServQueryRunner.java

@@ -371,6 +371,7 @@ private void doIt()
                     // and restrict to forward only so that client memory usage is minimal since
                     // we are only interested in reading the ResultSet once
 
+		    connection.setAutoCommit(false);
                     pstmt = connection.prepareStatement(sql);
                     pstmt.setFetchSize(1000);
                     pstmt.setFetchDirection(ResultSet.FETCH_FORWARD);

src/main/java/org/opencadc/tap/impl/ResultStoreImpl.java

@@ -103,8 +103,7 @@ public class ResultStoreImpl implements ResultStore {
     public URL put(final ResultSet resultSet,
                    final TableWriter<ResultSet> resultSetTableWriter)
         throws IOException {
-        OutputStream os;
-        os = getOutputStream();
+        OutputStream os = getOutputStream();
         resultSetTableWriter.write(resultSet, os);
         os.close();
         return getURL();
@@ -113,8 +112,7 @@ public URL put(final ResultSet resultSet,
     @Override
     public URL put(Throwable throwable, TableWriter tableWriter)
         throws IOException {
-        OutputStream os;
-        os = getOutputStream();
+        OutputStream os = getOutputStream();
         tableWriter.write(throwable, os);
         os.close();
         return getURL();
@@ -124,8 +122,7 @@ public URL put(Throwable throwable, TableWriter tableWriter)
     public URL put(final ResultSet resultSet,
                    final TableWriter<ResultSet> resultSetTableWriter,
                    final Integer integer) throws IOException {
-        OutputStream os;
-        os = getOutputStream();
+        OutputStream os = getOutputStream();
 
         if (integer == null) {
             resultSetTableWriter.write(resultSet, os);