On 11/26/08, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> "Merlin Moncure" <mmoncure(at)gmail(dot)com> writes:
> > On Mon, Nov 24, 2008 at 10:25 AM, Marko Kreen <markokr(at)gmail(dot)com> wrote:
> >> IOW, DISCARD ALL should be functionally equivalent to backend exit.
> > Having done a lot of work with advisory locks, I support this change.
> > Advisory locks are essentially session scoped objects like prepared
> > statements or notifies. It's only natural to clean them up in the
> > same way.
> > That said, I don't think this should be backpatched to 8.3.
> Done but not back-patched.
I think this should be back-patched as well:
- The fact that disconnect will clean up used resources has been
always true, thus most clients assume at some level.
- DISCARD ALL was new feature in 8.3. It is highly doubtful some
adv-locks using project has managed to hard-code dependency on
buggy behaviour of DISCARD.
- The bug was reported by regular user who encountered deadlocks
on 8.3 because of it.
In response to
pgsql-hackers by date
|Next:||From: Michael Meskes||Date: 2008-11-26 16:31:51|
|Subject: Re: Re: Updated interval patches - ECPG [was,intervalstyle....]|
|Previous:||From: Pavel Stehule||Date: 2008-11-26 15:58:11|
|Subject: Re: what is necessary for filling SysCache?|