[next] Avoid expensive index recreation

appinfo/info.xml

@@ -58,7 +58,6 @@
   <repair-steps>
     <pre-migration>
       <step>OCA\Polls\Migration\RepairSteps\RemoveObsoleteMigrations</step>
-      <step>OCA\Polls\Migration\RepairSteps\RemoveIndices</step>
     </pre-migration>
     <post-migration>
       <step>OCA\Polls\Migration\RepairSteps\DropOrphanedTables</step>

lib/Command/Db/Rebuild.php

@@ -1,9 +1,9 @@
 <?php
 
 declare(strict_types=1);
-/**
- * SPDX-FileCopyrightText: 2021 Nextcloud contributors
- * SPDX-License-Identifier: AGPL-3.0-or-later
+/**
+ * SPDX-FileCopyrightText: 2021 Nextcloud contributors
+ * SPDX-License-Identifier: AGPL-3.0-or-later
  */
 
 namespace OCA\Polls\Command\Db;
@@ -23,6 +23,12 @@ class Rebuild extends Command {
 	protected array $operationHints = [
 		'All polls tables will get checked against the current schema.',
 		'NO data migration will be executed, so make sure you have a backup of your database.',
+		'',
+		'*****************************',
+		'**    Please understand    **',
+		'*****************************',
+		'The process will also recreate all indices and foreign key constraints.',
+		'This can lead to a database performance impact on the app after the recreation is done.',
 	];
 
 	public function __construct(

lib/Db/IndexManager.php

@@ -73,7 +73,7 @@ public function createForeignKeyConstraints(): array {
 	/**
 	 * add an on delete fk contraint
 	 *
-	 * @param string $parentTableName name of reffered table
+	 * @param string $parentTableName name of referred table
 	 * @param string $childTableName name of referring table
 	 * @return string log message
 	 */