From 63725f7edba44438faaafef013b46a21da0e9164 Mon Sep 17 00:00:00 2001 From: Shane Synan Date: Tue, 29 Jan 2019 00:07:10 -0500 Subject: [PATCH] core: Remove redundant updateSchemaVersion() call Remove the redundant updateSchemaVersion() call at the end of schema migrations. This is already called within the final iteration of the loop. Cleans up a leftover from f10304a35af0a7a4f8b812e467e69287d358ce7c --- src/core/abstractsqlstorage.cpp | 9 --------- 1 file changed, 9 deletions(-) diff --git a/src/core/abstractsqlstorage.cpp b/src/core/abstractsqlstorage.cpp index c00ab62c..625037ac 100644 --- a/src/core/abstractsqlstorage.cpp +++ b/src/core/abstractsqlstorage.cpp @@ -324,15 +324,6 @@ bool AbstractSqlStorage::upgradeDb() } } - // Update the schema version for the final step. Split this out to offer more informative - // logging (though setting schema version really should not fail). - if (!updateSchemaVersion(schemaVersion())) { - // Updating the final schema version failed, bail out - qCritical() << "Unable to upgrade Logging Backend! Setting final schema version" - << schemaVersion() << "failed."; - return false; - } - // If we made it here, everything seems to have worked! return true; } -- 2.20.1