On Connection initialisation, setAutoCommit() mode later (after clientInfo etc)

This change means that autoCommit true will be active while clientInfo is being
set and initSql executed.

Specifically, clientInfo is not supposed to need autoCommit true (or an explicit
commit) but it does seem so for Postgres, and for a very idle database using
autoCommit false and setting clientInfo ApplicationName.

Author: rbygrave

ebean-datasource/src/main/java/io/ebean/datasource/pool/ConnectionPool.java

@@ -439,7 +439,6 @@ private Connection initConnection(Connection conn) throws SQLException {
     if (connectionInitializer != null) {
       connectionInitializer.preInitialize(conn);
     }
-    conn.setAutoCommit(autoCommit);
     // isolation level is set globally for all connections (at least for H2) and
     // you will need admin rights - so we do not change it, if it already matches.
     if (conn.getTransactionIsolation() != transactionIsolation) {
@@ -475,6 +474,7 @@ private Connection initConnection(Connection conn) throws SQLException {
         }
       }
     }
+    conn.setAutoCommit(autoCommit);
     if (connectionInitializer != null) {
       connectionInitializer.postInitialize(conn);
     }