Peter Eisentraut <peter_e@gmx.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