Skip to content

Commit 9dcf3c7

Browse files
committed
database: Simplify trigger_versions_set_updated_at trigger on the versions table
The only mutable columns in this table are `yanked` and `downloads`, but the trigger should specifically not be used when the `downloads` column changes. That means we're only left with the `yanked` column and can be more specific in the trigger condition.
1 parent e41f2a4 commit 9dcf3c7

File tree

2 files changed

+18
-0
lines changed

2 files changed

+18
-0
lines changed
Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
-- remove the new trigger
2+
DROP TRIGGER trigger_versions_set_updated_at ON versions;
3+
4+
-- add the old trigger again
5+
CREATE TRIGGER trigger_versions_set_updated_at
6+
BEFORE UPDATE
7+
ON versions
8+
FOR EACH ROW
9+
EXECUTE PROCEDURE set_updated_at_ignore_downloads();
Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
-- remove the old trigger
2+
DROP TRIGGER trigger_versions_set_updated_at ON versions;
3+
4+
-- add the new trigger only for the `yanked` column
5+
CREATE TRIGGER trigger_versions_set_updated_at
6+
BEFORE UPDATE OF yanked
7+
ON versions
8+
FOR EACH ROW
9+
EXECUTE PROCEDURE set_updated_at();

0 commit comments

Comments
 (0)