fix: solve perf issue in log's query (upgrade scripts+debug)

theus77 · Oct 31, 2024 · bcb83b1 · bcb83b1
1 parent f8e3871
commit bcb83b1
Show file tree

Hide file tree

Showing 5 changed files with 141 additions and 1 deletion.
diff --git a/EMS/common-bundle/src/Entity/Log.php b/EMS/common-bundle/src/Entity/Log.php
@@ -9,7 +9,7 @@
 use Ramsey\Uuid\UuidInterface;
 
 /**
- * @ORM\Table(name="log_message",indexes={@Index(name="channel_ouuid_idx", columns={"channel", "ouuid"})})
+ * @ORM\Table(name="log_message",indexes={@ORM\Index(name="channel_ouuid_idx", columns={"channel", "ouuid"})})
  *
  * @ORM\Entity
  *

diff --git a/EMS/core-bundle/src/Resources/DoctrineMigrations/pdo_mysql/Version20241031110947.php b/EMS/core-bundle/src/Resources/DoctrineMigrations/pdo_mysql/Version20241031110947.php
@@ -0,0 +1,35 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Application\Migrations;
+
+use Doctrine\DBAL\Platforms\MySQLPlatform;
+use Doctrine\DBAL\Schema\Schema;
+use Doctrine\Migrations\AbstractMigration;
+
+final class Version20241031110947 extends AbstractMigration
+{
+    public function getDescription(): string
+    {
+        return 'Add a non-unique channel/ouuid index in logs table';
+    }
+
+    public function up(Schema $schema): void
+    {
+        $this->abortIf(
+            !$this->connection->getDatabasePlatform() instanceof MySQLPlatform,
+            "Migration can only be executed safely on '\Doctrine\DBAL\Platforms\MySQLPlatform'."
+        );
+        $this->addSql('CREATE INDEX channel_ouuid_idx ON log_message (channel, ouuid)');
+    }
+
+    public function down(Schema $schema): void
+    {
+        $this->abortIf(
+            !$this->connection->getDatabasePlatform() instanceof MySQLPlatform,
+            "Migration can only be executed safely on '\Doctrine\DBAL\Platforms\MySQLPlatform'."
+        );
+        $this->addSql('DROP INDEX channel_ouuid_idx');
+    }
+}
diff --git a/EMS/core-bundle/src/Resources/DoctrineMigrations/pdo_pgsql/Version20241031110947.php b/EMS/core-bundle/src/Resources/DoctrineMigrations/pdo_pgsql/Version20241031110947.php
@@ -0,0 +1,35 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Application\Migrations;
+
+use Doctrine\DBAL\Platforms\PostgreSQLPlatform;
+use Doctrine\DBAL\Schema\Schema;
+use Doctrine\Migrations\AbstractMigration;
+
+final class Version20241031110947 extends AbstractMigration
+{
+    public function getDescription(): string
+    {
+        return 'Add a non-unique channel/ouuid index in logs table';
+    }
+
+    public function up(Schema $schema): void
+    {
+        $this->abortIf(
+            !$this->connection->getDatabasePlatform() instanceof PostgreSQLPlatform,
+            "Migration can only be executed safely on '\Doctrine\DBAL\Platforms\PostgreSQLPlatform'."
+        );
+        $this->addSql('CREATE INDEX channel_ouuid_idx ON log_message (channel, ouuid)');
+    }
+
+    public function down(Schema $schema): void
+    {
+        $this->abortIf(
+            !$this->connection->getDatabasePlatform() instanceof PostgreSQLPlatform,
+            "Migration can only be executed safely on '\Doctrine\DBAL\Platforms\PostgreSQLPlatform'."
+        );
+        $this->addSql('DROP INDEX channel_ouuid_idx');
+    }
+}
diff --git a/elasticms-web/migrations/pdo_mysql/Version20241031110947.php b/elasticms-web/migrations/pdo_mysql/Version20241031110947.php
@@ -0,0 +1,35 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Application\Migrations;
+
+use Doctrine\DBAL\Platforms\MySQLPlatform;
+use Doctrine\DBAL\Schema\Schema;
+use Doctrine\Migrations\AbstractMigration;
+
+final class Version20241031110947 extends AbstractMigration
+{
+    public function getDescription(): string
+    {
+        return 'Add a non-unique channel/ouuid index in logs table';
+    }
+
+    public function up(Schema $schema): void
+    {
+        $this->abortIf(
+            !$this->connection->getDatabasePlatform() instanceof MySQLPlatform,
+            "Migration can only be executed safely on '\Doctrine\DBAL\Platforms\MySQLPlatform'."
+        );
+        $this->addSql('CREATE INDEX channel_ouuid_idx ON log_message (channel, ouuid)');
+    }
+
+    public function down(Schema $schema): void
+    {
+        $this->abortIf(
+            !$this->connection->getDatabasePlatform() instanceof MySQLPlatform,
+            "Migration can only be executed safely on '\Doctrine\DBAL\Platforms\MySQLPlatform'."
+        );
+        $this->addSql('DROP INDEX channel_ouuid_idx');
+    }
+}
diff --git a/elasticms-web/migrations/pdo_pgsql/Version20241031110947.php b/elasticms-web/migrations/pdo_pgsql/Version20241031110947.php
@@ -0,0 +1,35 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Application\Migrations;
+
+use Doctrine\DBAL\Platforms\PostgreSQLPlatform;
+use Doctrine\DBAL\Schema\Schema;
+use Doctrine\Migrations\AbstractMigration;
+
+final class Version20241031110947 extends AbstractMigration
+{
+    public function getDescription(): string
+    {
+        return 'Add a non-unique channel/ouuid index in logs table';
+    }
+
+    public function up(Schema $schema): void
+    {
+        $this->abortIf(
+            !$this->connection->getDatabasePlatform() instanceof PostgreSQLPlatform,
+            "Migration can only be executed safely on '\Doctrine\DBAL\Platforms\PostgreSQLPlatform'."
+        );
+        $this->addSql('CREATE INDEX channel_ouuid_idx ON log_message (channel, ouuid)');
+    }
+
+    public function down(Schema $schema): void
+    {
+        $this->abortIf(
+            !$this->connection->getDatabasePlatform() instanceof PostgreSQLPlatform,
+            "Migration can only be executed safely on '\Doctrine\DBAL\Platforms\PostgreSQLPlatform'."
+        );
+        $this->addSql('DROP INDEX channel_ouuid_idx');
+    }
+}