Re: [HACKERS] Re: [COMMITTERS] pgsql/src/backend/utils/adt (ri_triggers.c)

[Hiroshi Inoue]

Tom Lane wrote: > [EMAIL PROTECTED] writes: > > Update of /home/projects/pgsql/cvsroot/pgsql/src/backend/utils/adt > > Modified Files: > > ri_triggers.c > > keep relations open until they are no longer needed. > > Something that's been bothering me for a good while about ri_triggers > is that it opens the relations without any lock to begin with. > That can't possibly be safe, can it? Opening relations with no lock seems illegal to me. Though I have no evidence that it does wrong thing in ri_triggers.c,it seems that we had better acquire an AccessShareLock on trial. I sometimes see SEGV error around ri stuff and I've doubted opening relations with no lock. However the cause was different from it. Hiroshi Inoue