Merge pull request #494 from wikimedia/fix-pdo-init

PDO: Move online schema creation behind feature flag

Author: glensc

config/config.default.php

@@ -18,6 +18,7 @@
         'pass' => getenv('XHGUI_PDO_PASS') ?: null,
         'table' => getenv('XHGUI_PDO_TABLE') ?: 'results',
         'tableWatch' => getenv('XHGUI_PDO_TABLE_WATCHES') ?: 'watches',
+        'initSchema' => getenv('XHGUI_PDO_INITSCHEMA') ?: 'true',
     ],
 
     // Database options for MongoDB.

src/Db/PdoRepository.php

@@ -34,7 +34,6 @@ public function __construct(PDO $pdo, string $driverName, string $table, string
         $this->driverName = $driverName;
         $this->table = sprintf('"%s"', $table);
         $this->tableWatches = sprintf('"%s"', $tableWatch);
-        $this->initSchema();
     }
 
     public function getLatest(): array

src/ServiceProvider/PdoStorageProvider.php

@@ -48,12 +48,16 @@ public function register(Container $app): void
         };
 
         $app[PdoRepository::class] = static function ($app) {
-            return new PdoRepository(
+            $repo = new PdoRepository(
                 $app['pdo'],
                 $app['pdo.driver'],
                 $app['config']['pdo']['table'],
                 $app['config']['pdo']['tableWatch']
             );
+            if ($app['config']['pdo']['initSchema'] === 'true') {
+                $repo->initSchema();
+            }
+            return $repo;
         };
 
         $app['searcher.pdo'] = static function ($app) {