Re: SQL:2011 Application Time Update & Delete - Mailing list pgsql-hackers

From SATYANARAYANA NARLAPURAM
Subject Re: SQL:2011 Application Time Update & Delete
Date
Msg-id CAHg+QDd74fnd4obCRMqVS0AVWf=cSFH=Cv7trTJWgm+_bhTK6w@mail.gmail.com
Whole thread
In response to Re: SQL:2011 Application Time Update & Delete  (SATYANARAYANA NARLAPURAM <satyanarlapuram@gmail.com>)
Responses Re: SQL:2011 Application Time Update & Delete
List pgsql-hackers
Hi Paul, Peter,

I found a Server crash when using UPDATE ... FOR PORTION OF or DELETE ... FOR PORTION OF on a view that has INSTEAD OF triggers.

Repro:

CREATE TABLE t (id INT, valid_at daterange, val INT);
INSERT INTO t VALUES (1, '[2026-01-01,2026-12-31)', 100);
CREATE VIEW v AS SELECT * FROM t;

CREATE FUNCTION v_trig() RETURNS trigger LANGUAGE plpgsql AS $$
BEGIN
    UPDATE t SET val = NEW.val WHERE id = OLD.id;
    RETURN NEW;
END;
$$;
CREATE TRIGGER trg INSTEAD OF UPDATE ON v
    FOR EACH ROW EXECUTE FUNCTION v_trig();

-- This crashes the server:
UPDATE v FOR PORTION OF valid_at FROM '2026-04-01' TO '2026-08-01'
    SET val = 999 WHERE id = 1;

I am thinking we should just reject this case. Attached a draft patch to fix the issue. 

Thanks,
Satya

pgsql-hackers by date:

Previous
From: Daniil Davydov
Date:
Subject: Re: test_autovacuum/001_parallel_autovacuum is broken
Next
From: Masahiko Sawada
Date:
Subject: Re: test_autovacuum/001_parallel_autovacuum is broken