Add an INFO level log message when non-transactional migrations are run

Author: rob-bygrave

ebean-migration/src/main/java/io/ebean/migration/MigrationRunner.java

@@ -93,7 +93,7 @@ protected void run(Connection connection, boolean checkStateOnly) {
         return;
       }
 
-      long start = System.currentTimeMillis();
+      long startMs = System.currentTimeMillis();
       connection.setAutoCommit(false);
       MigrationPlatform platform = derivePlatformName(migrationConfig, connection);
       new MigrationSchema(migrationConfig, connection).createAndSetIfNeeded();
@@ -105,9 +105,12 @@ protected void run(Connection connection, boolean checkStateOnly) {
         runMigrations(migrations, table, checkStateOnly);
         connection.commit();
         if (!checkStateOnly) {
-          long exeMillis = System.currentTimeMillis() - start;
-          log.log(INFO, "DB migrations completed in {0}ms - executed:{1} size:{2}", exeMillis, table.count(), migrations.size());
-          table.runNonTransactional();
+          long commitMs = System.currentTimeMillis();
+          log.log(INFO, "DB migrations completed in {0}ms - executed:{1} totalMigrations:{2}", (commitMs - startMs), table.count(), migrations.size());
+          int countNonTransactional = table.runNonTransactional();
+          if (countNonTransactional > 0) {
+            log.log(INFO, "Non-transactional DB migrations completed in {0}ms - executed:{1}", (System.currentTimeMillis() - commitMs), countNonTransactional);
+          }
         }
       } finally {
         table.unlockMigrationTable();

ebean-migration/src/main/java/io/ebean/migration/runner/MigrationScriptRunner.java

@@ -34,10 +34,11 @@ void runScript(String content, String scriptName) throws SQLException {
     nonTransactional.addAll(runner.runAll(content, connection));
   }
 
-  public void runNonTransactional() {
+  int runNonTransactional() {
     if (!nonTransactional.isEmpty()) {
       DdlRunner runner = new DdlRunner(false, "Non-transactional DDL", platform.ddlDetect());
       runner.runNonTransactional(connection, nonTransactional);
     }
+    return nonTransactional.size();
   }
 }

ebean-migration/src/main/java/io/ebean/migration/runner/MigrationTable.java

@@ -554,8 +554,8 @@ private void runRepeatableInit(List<LocalMigrationResource> localVersions) throw
    * as such the migration isn't truely atomic - the migration can run and
    * complete and the non-transactional statements fail.
    */
-  public void runNonTransactional() {
-    scriptRunner.runNonTransactional();
+  public int runNonTransactional() {
+    return scriptRunner.runNonTransactional();
   }
 
   public int count() {