chore: add missing ON DELETE CASCADE to keto_relation_tuples (#1264)

ory · Feb 28, 2023 · 0dd2ba9 · 0dd2ba9
1 parent 4eab4b5
commit 0dd2ba9
Show file tree

Hide file tree

Showing 4 changed files with 229 additions and 0 deletions.
diff --git a/...ce/sql/migrations/sql/20230228091200000000_add-on-delete-cascade-to-relationship.down.sql b/...ce/sql/migrations/sql/20230228091200000000_add-on-delete-cascade-to-relationship.down.sql
@@ -0,0 +1,7 @@
+ALTER TABLE
+  keto_relation_tuples DROP CONSTRAINT keto_relation_tuples_nid_fk;
+
+ALTER TABLE
+  keto_relation_tuples
+ADD
+  CONSTRAINT keto_relation_tuples_uuid_nid_fk FOREIGN KEY (nid) REFERENCES networks (id);
diff --git a/...migrations/sql/20230228091200000000_add-on-delete-cascade-to-relationship.sqlite.down.sql b/...migrations/sql/20230228091200000000_add-on-delete-cascade-to-relationship.sqlite.down.sql
@@ -0,0 +1,109 @@
+ALTER TABLE
+  keto_relation_tuples RENAME TO _keto_relation_tuples;
+
+DROP INDEX keto_relation_tuples_uuid_subject_ids_idx;
+
+DROP INDEX keto_relation_tuples_uuid_subject_sets_idx;
+
+DROP INDEX keto_relation_tuples_uuid_full_idx;
+
+DROP INDEX keto_relation_tuples_uuid_reverse_subject_ids_idx;
+
+DROP INDEX keto_relation_tuples_uuid_reverse_subject_sets_idx;
+
+CREATE TABLE keto_relation_tuples (
+  shard_id UUID NOT NULL,
+  nid UUID NOT NULL,
+  namespace VARCHAR(200) NOT NULL,
+  object UUID NOT NULL,
+  relation VARCHAR(64) NOT NULL,
+  subject_id UUID NULL,
+  subject_set_namespace VARCHAR(200) NULL,
+  subject_set_object UUID NULL,
+  subject_set_relation VARCHAR(64) NULL,
+  commit_time TIMESTAMP NOT NULL,
+  PRIMARY KEY (shard_id, nid),
+  CONSTRAINT keto_relation_tuples_uuid_nid_fk FOREIGN KEY (nid) REFERENCES networks (id),
+  CONSTRAINT chk_keto_rt_uuid_subject_type CHECK (
+    (
+      (
+        (
+          (subject_id IS NULL)
+          AND (subject_set_namespace IS NOT NULL)
+        )
+        AND (subject_set_object IS NOT NULL)
+      )
+      AND (subject_set_relation IS NOT NULL)
+    )
+    OR (
+      (
+        (
+          (subject_id IS NOT NULL)
+          AND (subject_set_namespace IS NULL)
+        )
+        AND (subject_set_object IS NULL)
+      )
+      AND (subject_set_relation IS NULL)
+    )
+  )
+);
+
+CREATE INDEX keto_relation_tuples_uuid_subject_ids_idx ON keto_relation_tuples (nid, namespace, object, relation, subject_id)
+WHERE
+  (
+    (subject_set_namespace IS NULL)
+    AND (subject_set_object IS NULL)
+  )
+  AND (subject_set_relation IS NULL);
+
+CREATE INDEX keto_relation_tuples_uuid_subject_sets_idx ON keto_relation_tuples (
+  nid,
+  namespace,
+  object,
+  relation,
+  subject_set_namespace,
+  subject_set_object,
+  subject_set_relation
+)
+WHERE
+  subject_id IS NULL;
+
+CREATE INDEX keto_relation_tuples_uuid_full_idx ON keto_relation_tuples (
+  nid,
+  namespace,
+  object,
+  relation,
+  subject_id,
+  subject_set_namespace,
+  subject_set_object,
+  subject_set_relation,
+  commit_time
+);
+
+CREATE INDEX keto_relation_tuples_uuid_reverse_subject_ids_idx ON keto_relation_tuples (nid, subject_id, relation, namespace)
+WHERE
+  (
+    (subject_set_namespace IS NULL)
+    AND (subject_set_object IS NULL)
+  )
+  AND (subject_set_relation IS NULL);
+
+CREATE INDEX keto_relation_tuples_uuid_reverse_subject_sets_idx ON keto_relation_tuples (
+  nid,
+  subject_set_namespace,
+  subject_set_object,
+  subject_set_relation,
+  relation,
+  namespace
+)
+WHERE
+  subject_id IS NULL;
+
+INSERT INTO
+  keto_relation_tuples
+SELECT
+  *
+FROM
+  _keto_relation_tuples;
+
+DROP TABLE _keto_relation_tuples;
diff --git a/...l/migrations/sql/20230228091200000000_add-on-delete-cascade-to-relationship.sqlite.up.sql b/...l/migrations/sql/20230228091200000000_add-on-delete-cascade-to-relationship.sqlite.up.sql
@@ -0,0 +1,109 @@
+ALTER TABLE
+  keto_relation_tuples RENAME TO _keto_relation_tuples;
+
+DROP INDEX keto_relation_tuples_uuid_subject_ids_idx;
+
+DROP INDEX keto_relation_tuples_uuid_subject_sets_idx;
+
+DROP INDEX keto_relation_tuples_uuid_full_idx;
+
+DROP INDEX keto_relation_tuples_uuid_reverse_subject_ids_idx;
+
+DROP INDEX keto_relation_tuples_uuid_reverse_subject_sets_idx;
+
+CREATE TABLE keto_relation_tuples (
+  shard_id UUID NOT NULL,
+  nid UUID NOT NULL,
+  namespace VARCHAR(200) NOT NULL,
+  object UUID NOT NULL,
+  relation VARCHAR(64) NOT NULL,
+  subject_id UUID NULL,
+  subject_set_namespace VARCHAR(200) NULL,
+  subject_set_object UUID NULL,
+  subject_set_relation VARCHAR(64) NULL,
+  commit_time TIMESTAMP NOT NULL,
+  PRIMARY KEY (shard_id, nid),
+  CONSTRAINT keto_relation_tuples_nid_fk FOREIGN KEY (nid) REFERENCES networks (id) ON UPDATE RESTRICT ON DELETE CASCADE,
+  CONSTRAINT chk_keto_rt_uuid_subject_type CHECK (
+    (
+      (
+        (
+          (subject_id IS NULL)
+          AND (subject_set_namespace IS NOT NULL)
+        )
+        AND (subject_set_object IS NOT NULL)
+      )
+      AND (subject_set_relation IS NOT NULL)
+    )
+    OR (
+      (
+        (
+          (subject_id IS NOT NULL)
+          AND (subject_set_namespace IS NULL)
+        )
+        AND (subject_set_object IS NULL)
+      )
+      AND (subject_set_relation IS NULL)
+    )
+  )
+);
+
+CREATE INDEX keto_relation_tuples_uuid_subject_ids_idx ON keto_relation_tuples (nid, namespace, object, relation, subject_id)
+WHERE
+  (
+    (subject_set_namespace IS NULL)
+    AND (subject_set_object IS NULL)
+  )
+  AND (subject_set_relation IS NULL);
+
+CREATE INDEX keto_relation_tuples_uuid_subject_sets_idx ON keto_relation_tuples (
+  nid,
+  namespace,
+  object,
+  relation,
+  subject_set_namespace,
+  subject_set_object,
+  subject_set_relation
+)
+WHERE
+  subject_id IS NULL;
+
+CREATE INDEX keto_relation_tuples_uuid_full_idx ON keto_relation_tuples (
+  nid,
+  namespace,
+  object,
+  relation,
+  subject_id,
+  subject_set_namespace,
+  subject_set_object,
+  subject_set_relation,
+  commit_time
+);
+
+CREATE INDEX keto_relation_tuples_uuid_reverse_subject_ids_idx ON keto_relation_tuples (nid, subject_id, relation, namespace)
+WHERE
+  (
+    (subject_set_namespace IS NULL)
+    AND (subject_set_object IS NULL)
+  )
+  AND (subject_set_relation IS NULL);
+
+CREATE INDEX keto_relation_tuples_uuid_reverse_subject_sets_idx ON keto_relation_tuples (
+  nid,
+  subject_set_namespace,
+  subject_set_object,
+  subject_set_relation,
+  relation,
+  namespace
+)
+WHERE
+  subject_id IS NULL;
+
+INSERT INTO
+  keto_relation_tuples
+SELECT
+  *
+FROM
+  _keto_relation_tuples;
+
+DROP TABLE _keto_relation_tuples;
diff --git a/...ence/sql/migrations/sql/20230228091200000000_add-on-delete-cascade-to-relationship.up.sql b/...ence/sql/migrations/sql/20230228091200000000_add-on-delete-cascade-to-relationship.up.sql
@@ -0,0 +1,4 @@
+ALTER TABLE
+  keto_relation_tuples DROP CONSTRAINT keto_relation_tuples_uuid_nid_fk,
+ADD
+  CONSTRAINT keto_relation_tuples_nid_fk FOREIGN KEY (nid) REFERENCES networks (id) ON UPDATE RESTRICT ON DELETE CASCADE;