TODO item on triggers

Started by Bruce Momjianover 23 years ago3 messageshackers
Jump to latest
#1Bruce Momjian
bruce@momjian.us

Is this item completed? It sure looks like it:

* Make triggers refer to columns by number, not name

test=> \d pg_trigger
Table "pg_catalog.pg_trigger"
Column | Type | Modifiers
----------------+------------+-----------
tgrelid | oid | not null
tgname | name | not null
tgfoid | oid | not null
tgtype | smallint | not null
tgenabled | boolean | not null
tgisconstraint | boolean | not null
tgconstrname | name | not null
tgconstrrelid | oid | not null
tgdeferrable | boolean | not null
tginitdeferred | boolean | not null
tgnargs | smallint | not null
tgattr | int2vector | not null
tgargs | bytea |

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 359-1001
  +  If your life is a hard drive,     |  13 Roberts Road
  +  Christ can be your backup.        |  Newtown Square, Pennsylvania 19073
#2Tom Lane
tgl@sss.pgh.pa.us
In reply to: Bruce Momjian (#1)
Re: TODO item on triggers

Bruce Momjian <pgman@candle.pha.pa.us> writes:

Is this item completed? It sure looks like it:
* Make triggers refer to columns by number, not name

It is not necessary anymore. The triggers still use names, but there's
code in ALTER...RENAME to fix the trigger parameters. I'm perfectly
happy with that solution and see no need to do what the TODO item
suggests.

regards, tom lane

#3Bruce Momjian
bruce@momjian.us
In reply to: Tom Lane (#2)
Re: TODO item on triggers

Tom Lane wrote:

Bruce Momjian <pgman@candle.pha.pa.us> writes:

Is this item completed? It sure looks like it:
* Make triggers refer to columns by number, not name

It is not necessary anymore. The triggers still use names, but there's
code in ALTER...RENAME to fix the trigger parameters. I'm perfectly
happy with that solution and see no need to do what the TODO item
suggests.

OK, item removed, again. ;-)

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 359-1001
  +  If your life is a hard drive,     |  13 Roberts Road
  +  Christ can be your backup.        |  Newtown Square, Pennsylvania 19073