Re: Triggers on columns

From: Itagaki Takahiro <itagaki(dot)takahiro(at)oss(dot)ntt(dot)co(dot)jp>
To: Peter Eisentraut <peter_e(at)gmx(dot)net>
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: Triggers on columns
Date: 2009-10-14 04:34:31
Message-ID: 20091014132422.B1E5.52131E4D@oss.ntt.co.jp
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers


Peter Eisentraut <peter_e(at)gmx(dot)net> wrote:

> I think the patch is almost ready now. One remaining issue is, in
> TriggerEnabled() you apparently check the column list only if it is a
> row trigger. But columns are supported for statement triggers as well
> per SQL standard. Check please.

I added it. I've missed it because I tried to implement column
trigger based on value-comparison, but now we check conditions
based on a target-list. Statement column triggers are reasonable.

diff-of-column-trigger.patch is a diff from the previous column-trigger.patch.

column-trigger_20091014.patch is a full patch to the HEAD.

Regards,
---
ITAGAKI Takahiro
NTT Open Source Software Center

Attachment Content-Type Size
column-trigger_20091014.patch application/octet-stream 38.7 KB
diff-of-column-trigger.patch application/octet-stream 8.3 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message KaiGai Kohei 2009-10-14 05:34:32 Re: [PATCH] Largeobject access controls
Previous Message KaiGai Kohei 2009-10-14 03:07:46 Reworks for Access Control facilities (r2363)