pleroma

commit: 9b45feae9b5102acb7b9431faa7eaaffcd7e634f
parent: 079266d40658ee7042f7bc8a855485069a474ef8
Author: feld <feld@feld.me>
Date:   Wed,  3 Jul 2019 16:54:47 +0000

Merge branch 'fix/migration-alter-citext-field-with-key' into 'develop'

Fix occasional migration failures

See merge request pleroma/pleroma!1373
Diffstat:
M
priv/repo/migrations/20170522160642_case_insensivtivity.exs
24
++++++++++++++++++------

1 file changed, 18 insertions(+), 6 deletions(-)
diff --git a/priv/repo/migrations/20170522160642_case_insensivtivity.exs b/priv/repo/migrations/20170522160642_case_insensivtivity.exs
@@ -1,19 +1,31 @@
 defmodule Pleroma.Repo.Migrations.CaseInsensivtivity do
   use Ecto.Migration
 
+  # Two-steps alters are intentional.
+  # When alter of 2 columns is done in a single operation,
+  # inconsistent failures happen because of index on `email` column.
+
   def up do
-    execute ("create extension if not exists citext")
+    execute("create extension if not exists citext")
+
+    alter table(:users) do
+      modify(:email, :citext)
+    end
+
     alter table(:users) do
-      modify :email, :citext
-      modify :nickname, :citext
+      modify(:nickname, :citext)
     end
   end
 
   def down do
     alter table(:users) do
-      modify :email, :string
-      modify :nickname, :string
+      modify(:email, :string)
     end
-    execute ("drop extension if exists citext")
+
+    alter table(:users) do
+      modify(:nickname, :string)
+    end
+
+    execute("drop extension if exists citext")
   end
 end