Re: [PATCHES] Hash Index Build Patch v2

2008-03-16 Thread Tom Lane
Tom Raney [EMAIL PROTECTED] writes: This revised version of our patch uses the function estimate_rel_size() from plancat.c to estimate the number of tuples in the parent relation. This method is an alternative to scanning the parent relation to estimate the number of tuples, as we did in

Re: [PATCHES] sinval.c / sinvaladt.c restructuring

2008-03-16 Thread Tom Lane
Alvaro Herrera [EMAIL PROTECTED] writes: One side effect of this change is that the call to SendPostmasterSignal now occurs after the lock has been released. ISTM this is a good idea on general principles (no syscalls in lwlocked code), but I'm wondering if I created a thundering hoard

Re: [PATCHES] [PERFORM] Very slow (2 tuples/second) sequential scan after bulk insert; speed returns to ~500 tuples/second after commit

2008-03-16 Thread Tom Lane
Heikki Linnakangas [EMAIL PROTECTED] writes: Yep, patch attached. I also changed xactGetCommittedChildren to return the original array instead of copying it, as Alvaro suggested. Applied with minor corrections (mostly comment fixes, but there were a couple of real mistakes).

Re: [PATCHES] [PERFORM] Very slow (2 tuples/second) sequential scan after bulk insert; speed returns to ~500 tuples/second after commit

2008-03-16 Thread Bruce Momjian
This has been applied by Tom. --- Heikki Linnakangas wrote: Tom Lane wrote: Heikki Linnakangas [EMAIL PROTECTED] writes: Elsewhere in our codebase where we use arrays that are enlarged as needed, we keep track of

Re: [PATCHES] updated hash functions for postgresql v1

2008-03-16 Thread Tom Lane
Kenneth Marshall [EMAIL PROTECTED] writes: Dear PostgreSQL Developers, This patch is a diff -c against the hashfunc.c from postgresql-8.3beta1. It's pretty obvious that this patch hasn't even been tested on a big-endian machine: + #ifndef WORS_BIGENDIAN However, why do we need two code paths

Re: [PATCHES] Suppress compiler warnings on mingw

2008-03-16 Thread ITAGAKI Takahiro
Peter Eisentraut [EMAIL PROTECTED] wrote: Then try using %lu and no casts. That should get rid of the warnings the proper way. Ok, I rewrote it to use %lu for format strings. Jeremy Drake [EMAIL PROTECTED] wrote: sizeof(DWORD) is always 4, even on 64-bit windows. sizeof(long) is also

[PATCHES] [4/4] Proposal of SE-PostgreSQL patches

2008-03-16 Thread Kohei KaiGai
[4/4] - sepostgresql-policy-8.4devel-3.patch This patch gives us the default security policy for SE-PostgreSQL. You can build it as a security policy module. It can be linked with the existing distributor's policy, and reloaded. -- OSS Platform Development Division, NEC KaiGai Kohei [EMAIL

Re: [PATCHES] [0/4] Proposal of SE-PostgreSQL patches

2008-03-16 Thread Kohei KaiGai
It seems to me some of SE-PostgreSQL patches are not delivered yet, although [3/4] and [4/4] were already done. Does anti-spam system caught my previous three messages? If necessary, I will send them again. Thanks, Kohei KaiGai wrote: The series of patches are the proposal of Security-Enhanced

[PATCHES] [0/4] Proposal of SE-PostgreSQL patches

2008-03-16 Thread Kohei KaiGai
The series of patches are the proposal of Security-Enhanced PostgreSQL (SE-PostgreSQL) for the upstreamed PostgreSQL 8.4 development cycle. [1/4] sepostgresql-pgace-8.4devel-3.patch provides PGACE (PostgreSQL Access Control Extension) framework [2/4]