Re: [PATCHES] pgstattuple locking fix

2007-10-22 Thread Tom Lane
"Heikki Linnakangas" <[EMAIL PROTECTED]> writes: > On second thought, we do call PageGetHeapFreeSpace without holding a > lock in heap_page_prune_opt as well, so it better be safe. Looking > closer at PageGetHeapFreeSpace, I think it is. The > return value can be bogus, of course. > That's worth n

Re: [PATCHES] pgstattuple locking fix

2007-10-22 Thread Tom Lane
"Heikki Linnakangas" <[EMAIL PROTECTED]> writes: > ITAGAKI Takahiro wrote: >> Tom Lane <[EMAIL PROTECTED]> wrote: >>> I'm thinking that having a pin on the buffer should be enough to >>> call PageGetHeapFreeSpace. >> >> Hmm... we might use pd_upper and pd_lower at different times, > No I think th

Re: [PATCHES] EXECUTE USING for plpgsql (for 8.4)

2007-10-22 Thread Bruce Momjian
This has been saved for the 8.4 release: http://momjian.postgresql.org/cgi-bin/pgpatches_hold --- Pavel Stehule wrote: > Hello > > this patch add USING clause into plpgsql EXECUTE statements. > > Proposal: > http:

Re: [PATCHES] Hash Index Build Patch v2

2007-10-22 Thread Bruce Momjian
This has been saved for the 8.4 release: http://momjian.postgresql.org/cgi-bin/pgpatches_hold --- Tom Raney wrote: > This revised version of our patch uses the function estimate_rel_size() > from plancat.c to estim

[PATCHES] EXECUTE USING for plpgsql (for 8.4)

2007-10-22 Thread Pavel Stehule
Hello this patch add USING clause into plpgsql EXECUTE statements. Proposal: http://archives.postgresql.org/pgsql-hackers/2007-10/msg00790.php I found, so dynamics statements are little bit faster with parameters, because we don't need call lot of in out/in functions. Mainly it is barier to SQL

Re: [PATCHES] pgstattuple locking fix

2007-10-22 Thread Heikki Linnakangas
Heikki Linnakangas wrote: > ITAGAKI Takahiro wrote: >> Tom Lane <[EMAIL PROTECTED]> wrote: >> >>> ITAGAKI Takahiro <[EMAIL PROTECTED]> writes: Here is a trivial fix of locking issue in pgstattuple(). >>> Hmm, is this really a bug, and if so how far back does it go? >>> I'm thinking that having

Re: [PATCHES] pgstattuple locking fix

2007-10-22 Thread Heikki Linnakangas
ITAGAKI Takahiro wrote: > Tom Lane <[EMAIL PROTECTED]> wrote: > >> ITAGAKI Takahiro <[EMAIL PROTECTED]> writes: >>> Here is a trivial fix of locking issue in pgstattuple(). >> Hmm, is this really a bug, and if so how far back does it go? >> I'm thinking that having a pin on the buffer should be en

Re: [PATCHES] pgstattuple locking fix

2007-10-22 Thread ITAGAKI Takahiro
Tom Lane <[EMAIL PROTECTED]> wrote: > ITAGAKI Takahiro <[EMAIL PROTECTED]> writes: > > Here is a trivial fix of locking issue in pgstattuple(). > > Hmm, is this really a bug, and if so how far back does it go? > I'm thinking that having a pin on the buffer should be enough to > call PageGetHeapFr