Mirrors/lemmy
2
0
Fork 0
mirror of https://github.com/LemmyNet/lemmy synced 2024-11-22 12:33:09 +00:00
Code Issues Projects Releases Packages Wiki Activity

remove performance-problematic and buggy duplicate site aggregates (#3732)

Browse source
This commit is contained in:
phiresky 2023-07-27 23:35:53 +02:00 committed by GitHub
parent d909f3455d
commit 1253a2a0d5
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 26 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
migrations/2023-07-26-222023_site-aggregates-one/down.sql Normal file
View file

@ -0,0 +1,11 @@
create or replace function site_aggregates_site()
returns trigger language plpgsql
as $$
begin
IF (TG_OP = 'INSERT') THEN
insert into site_aggregates (site_id) values (NEW.id);
ELSIF (TG_OP = 'DELETE') THEN
delete from site_aggregates where site_id = OLD.id;
END IF;
return null;
end $$;

15
migrations/2023-07-26-222023_site-aggregates-one/up.sql Normal file
View file

@ -0,0 +1,15 @@
create or replace function site_aggregates_site()
returns trigger language plpgsql
as $$
begin
-- we only ever want to have a single value in site_aggregate because the site_aggregate triggers update all rows in that table.
-- a cleaner check would be to insert it for the local_site but that would break assumptions at least in the tests
IF (TG_OP = 'INSERT') AND NOT EXISTS (select id from site_aggregates limit 1) THEN
insert into site_aggregates (site_id) values (NEW.id);
ELSIF (TG_OP = 'DELETE') THEN
delete from site_aggregates where site_id = OLD.id;
END IF;
return null;
end $$;
delete from site_aggregates a where not exists (select id from local_site s where s.site_id = a.site_id);