From 0a556f95c750904c1e40bb04c58904eeb12cb2c3 Mon Sep 17 00:00:00 2001
From: kirrg001 <katharina.irrgang@googlemail.com>
Date: Sun, 22 Jul 2018 14:03:59 +0200
Subject: [PATCH] Optimised logging for the new migration files

refs #9742
---
 .../migrations/versions/2.0/1-rename-amp-column.js     | 10 +++++++++-
 .../data/migrations/versions/2.0/2-update-posts.js     |  6 +++++-
 2 files changed, 14 insertions(+), 2 deletions(-)

diff --git a/core/server/data/migrations/versions/2.0/1-rename-amp-column.js b/core/server/data/migrations/versions/2.0/1-rename-amp-column.js
index 9096628eb1..a0546d645d 100644
--- a/core/server/data/migrations/versions/2.0/1-rename-amp-column.js
+++ b/core/server/data/migrations/versions/2.0/1-rename-amp-column.js
@@ -3,7 +3,9 @@ const common = require('../../../../lib/common'),
     columnNameOld = 'amp',
     columnNameNew = 'comment_id',
     message1 = `Renaming column ${columnNameOld} to ${columnNameNew}`,
-    message2 = `Renaming column ${columnNameNew} to ${columnNameOld}`;
+    message2 = `Renaming column ${columnNameNew} to ${columnNameOld}`,
+    message3 = `Renamed column ${columnNameOld} to ${columnNameNew}`,
+    message4 = `Renamed column ${columnNameNew} to ${columnNameOld}`;
 
 module.exports.up = function renameAmpColumn(options) {
     const connection = options.connection;
@@ -17,6 +19,9 @@ module.exports.up = function renameAmpColumn(options) {
                     t.renameColumn(columnNameOld, columnNameNew);
                 });
             }
+        })
+        .then(() => {
+            common.logging.info(message3);
         });
 };
 
@@ -32,5 +37,8 @@ module.exports.down = function renameCommentIdColumn(options) {
                     t.renameColumn(columnNameNew, columnNameOld);
                 });
             }
+        })
+        .then(() => {
+            common.logging.info(message4);
         });
 };
diff --git a/core/server/data/migrations/versions/2.0/2-update-posts.js b/core/server/data/migrations/versions/2.0/2-update-posts.js
index 04e7f88c75..79358dfcd3 100644
--- a/core/server/data/migrations/versions/2.0/2-update-posts.js
+++ b/core/server/data/migrations/versions/2.0/2-update-posts.js
@@ -2,7 +2,8 @@ const _ = require('lodash'),
     Promise = require('bluebird'),
     common = require('../../../../lib/common'),
     models = require('../../../../models'),
-    message1 = 'Updating post data (comment_id)';
+    message1 = 'Updating post data (comment_id)',
+    message2 = 'Updated post data (comment_id)';
 
 module.exports.config = {
     transaction: true
@@ -28,5 +29,8 @@ module.exports.up = function updatePosts(options) {
                     comment_id: post.id
                 }, _.merge({id: post.id}, localOptions));
             }, {concurrency: 100});
+        })
+        .then(() => {
+            common.logging.info(message2);
         });
 };