aboutsummaryrefslogtreecommitdiffstats
path: root/server/sonar-db-migration
diff options
context:
space:
mode:
authorPierre <pierre.guillot@sonarsource.com>2022-05-06 16:07:09 +0200
committersonartech <sonartech@sonarsource.com>2022-05-06 20:02:44 +0000
commit9f54cb9455149e9991e8ae780cf9a02cbeaf550b (patch)
tree2cc6042f43926c9c6e838559c4ed69fd541164f0 /server/sonar-db-migration
parenta196492676ffa49cc9df1f2c20cd916cacf5e212 (diff)
downloadsonarqube-9f54cb9455149e9991e8ae780cf9a02cbeaf550b.tar.gz
sonarqube-9f54cb9455149e9991e8ae780cf9a02cbeaf550b.zip
Revert "SONAR-16303 migrate plugin file hash to force reloading rules"
This reverts commit 7be1995f608831e9067b12da9be9706c3b04cd32.
Diffstat (limited to 'server/sonar-db-migration')
-rw-r--r--server/sonar-db-migration/src/main/java/org/sonar/server/platform/db/migration/version/v95/DbVersion95.java1
-rw-r--r--server/sonar-db-migration/src/main/java/org/sonar/server/platform/db/migration/version/v95/OverwritePluginFileHash.java42
-rw-r--r--server/sonar-db-migration/src/test/java/org/sonar/server/platform/db/migration/version/v95/OverwritePluginFileHashTest.java89
-rw-r--r--server/sonar-db-migration/src/test/resources/org/sonar/server/platform/db/migration/version/v95/OverwritePluginFileHashTest/schema.sql12
4 files changed, 0 insertions, 144 deletions
diff --git a/server/sonar-db-migration/src/main/java/org/sonar/server/platform/db/migration/version/v95/DbVersion95.java b/server/sonar-db-migration/src/main/java/org/sonar/server/platform/db/migration/version/v95/DbVersion95.java
index 2b2652dfeb6..bbdc14b3587 100644
--- a/server/sonar-db-migration/src/main/java/org/sonar/server/platform/db/migration/version/v95/DbVersion95.java
+++ b/server/sonar-db-migration/src/main/java/org/sonar/server/platform/db/migration/version/v95/DbVersion95.java
@@ -36,7 +36,6 @@ public class DbVersion95 implements DbVersion {
.add(6408, "Drop column DESCRIPTIONS from RULES table", DropRuleDescriptionColumn.class)
.add(6409, "Drop column CREATED_AT from RULES_METADATA table", DropRuleMetadataCreatedAtColumn.class)
.add(6410, "Drop column UPDATED_AT from RULES_METADATA table", DropRuleMetadataUpdatedAtColumn.class)
- .add(6411, "Overwrite plugin file hash to force reloading rules", OverwritePluginFileHash.class)
;
}
}
diff --git a/server/sonar-db-migration/src/main/java/org/sonar/server/platform/db/migration/version/v95/OverwritePluginFileHash.java b/server/sonar-db-migration/src/main/java/org/sonar/server/platform/db/migration/version/v95/OverwritePluginFileHash.java
deleted file mode 100644
index 578aca5a790..00000000000
--- a/server/sonar-db-migration/src/main/java/org/sonar/server/platform/db/migration/version/v95/OverwritePluginFileHash.java
+++ /dev/null
@@ -1,42 +0,0 @@
-/*
- * SonarQube
- * Copyright (C) 2009-2022 SonarSource SA
- * mailto:info AT sonarsource DOT com
- *
- * This program is free software; you can redistribute it and/or
- * modify it under the terms of the GNU Lesser General Public
- * License as published by the Free Software Foundation; either
- * version 3 of the License, or (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
- * Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with this program; if not, write to the Free Software Foundation,
- * Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
- */
-package org.sonar.server.platform.db.migration.version.v95;
-
-import com.google.common.annotations.VisibleForTesting;
-import java.sql.SQLException;
-import org.sonar.db.Database;
-import org.sonar.server.platform.db.migration.step.DataChange;
-import org.sonar.server.platform.db.migration.step.Upsert;
-
-public class OverwritePluginFileHash extends DataChange {
- @VisibleForTesting
- static final String OVERWRITE_HASH = "cccccccccccccccccccccccccccccccc";
-
- public OverwritePluginFileHash(Database db) {
- super(db);
- }
-
- @Override protected void execute(Context context) throws SQLException {
- Upsert upsert = context.prepareUpsert("update plugins set file_hash = ? ");
- upsert.setString(1, OVERWRITE_HASH);
- upsert.execute();
- upsert.commit();
- }
-}
diff --git a/server/sonar-db-migration/src/test/java/org/sonar/server/platform/db/migration/version/v95/OverwritePluginFileHashTest.java b/server/sonar-db-migration/src/test/java/org/sonar/server/platform/db/migration/version/v95/OverwritePluginFileHashTest.java
deleted file mode 100644
index 57f0e9bf3ac..00000000000
--- a/server/sonar-db-migration/src/test/java/org/sonar/server/platform/db/migration/version/v95/OverwritePluginFileHashTest.java
+++ /dev/null
@@ -1,89 +0,0 @@
-/*
- * SonarQube
- * Copyright (C) 2009-2022 SonarSource SA
- * mailto:info AT sonarsource DOT com
- *
- * This program is free software; you can redistribute it and/or
- * modify it under the terms of the GNU Lesser General Public
- * License as published by the Free Software Foundation; either
- * version 3 of the License, or (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
- * Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with this program; if not, write to the Free Software Foundation,
- * Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
- */
-package org.sonar.server.platform.db.migration.version.v95;
-
-import java.sql.SQLException;
-import java.util.HashMap;
-import java.util.Map;
-import org.junit.Rule;
-import org.junit.Test;
-import org.sonar.core.util.UuidFactory;
-import org.sonar.core.util.UuidFactoryFast;
-import org.sonar.db.CoreDbTester;
-import org.sonar.server.platform.db.migration.step.DataChange;
-
-import static org.apache.commons.lang.RandomStringUtils.randomAlphabetic;
-import static org.assertj.core.api.Assertions.assertThat;
-import static org.sonar.server.platform.db.migration.version.v95.OverwritePluginFileHash.OVERWRITE_HASH;
-
-public class OverwritePluginFileHashTest {
- private final UuidFactory uuidFactory = UuidFactoryFast.getInstance();
-
- @Rule
- public CoreDbTester db = CoreDbTester.createForSchema(OverwritePluginFileHashTest.class, "schema.sql");
-
- private final DataChange underTest = new OverwritePluginFileHash(db.database());
-
- @Test
- public void migration_overwrite_file_hash_on_all_plugins() throws SQLException {
- String pluginUuid1 = insertPlugin();
- String pluginUuid2 = insertPlugin();
-
- underTest.execute();
-
- assertPluginFileHashOverwrite(pluginUuid1);
- assertPluginFileHashOverwrite(pluginUuid2);
- }
-
- @Test
- public void migration_should_be_reentrant() throws SQLException {
- String pluginUuid1 = insertPlugin();
- String pluginUuid2 = insertPlugin();
-
- underTest.execute();
- // re-entrant
- underTest.execute();
-
- assertPluginFileHashOverwrite(pluginUuid1);
- assertPluginFileHashOverwrite(pluginUuid2);
- }
-
- private void assertPluginFileHashOverwrite(String pluginUuid) {
- String selectSql = String.format("select file_hash from plugins where uuid='%s'", pluginUuid);
- var selectResult = db.select(selectSql);
- assertThat(selectResult.get(0)).containsEntry("FILE_HASH", OVERWRITE_HASH);
- }
-
- private String insertPlugin() {
- Map<String, Object> map = new HashMap<>();
- String uuid = uuidFactory.create();
- map.put("UUID", uuid);
- map.put("KEE", randomAlphabetic(20));
- map.put("FILE_HASH", randomAlphabetic(32));
- map.put("CREATED_AT", System.currentTimeMillis());
- map.put("UPDATED_AT", System.currentTimeMillis());
- map.put("TYPE", "EXTERNAL");
- map.put("REMOVED", false);
- db.executeInsert("plugins", map);
-
- return uuid;
- }
-
-}
diff --git a/server/sonar-db-migration/src/test/resources/org/sonar/server/platform/db/migration/version/v95/OverwritePluginFileHashTest/schema.sql b/server/sonar-db-migration/src/test/resources/org/sonar/server/platform/db/migration/version/v95/OverwritePluginFileHashTest/schema.sql
deleted file mode 100644
index d51d6260dab..00000000000
--- a/server/sonar-db-migration/src/test/resources/org/sonar/server/platform/db/migration/version/v95/OverwritePluginFileHashTest/schema.sql
+++ /dev/null
@@ -1,12 +0,0 @@
-CREATE TABLE "PLUGINS"(
- "UUID" CHARACTER VARYING(40) NOT NULL,
- "KEE" CHARACTER VARYING(200) NOT NULL,
- "BASE_PLUGIN_KEY" CHARACTER VARYING(200),
- "FILE_HASH" CHARACTER VARYING(200) NOT NULL,
- "CREATED_AT" BIGINT NOT NULL,
- "UPDATED_AT" BIGINT NOT NULL,
- "TYPE" CHARACTER VARYING(10) NOT NULL,
- "REMOVED" BOOLEAN DEFAULT FALSE NOT NULL
-);
-ALTER TABLE "PLUGINS" ADD CONSTRAINT "PK_PLUGINS" PRIMARY KEY("UUID");
-CREATE UNIQUE INDEX "PLUGINS_KEY" ON "PLUGINS"("KEE" NULLS FIRST);