Re: [PATCHES] [HACKERS] roundoff problem in time datatype

2005-10-14 Thread Bruce Momjian
Patch applied, with documentation updates. --- Tom Lane wrote: > Inserting into a time field with limited precision rounds off, which > is good except for this case: > > regression=# select '23:59:59.9'::time(0); >time

Re: [PATCHES] [BUGS] BUG #1962: ECPG and VARCHAR

2005-10-14 Thread Michael Meskes
On Thu, Oct 13, 2005 at 09:53:14PM -0400, Bruce Momjian wrote: > Good catch! I have backpatched these fixes to the 8.0 and 7.4 branches > as you suggested, (identical) patches attached. Thanks Bruce. This was an oversight on my part. I should have committed to 8.0 branch too. I'm sorry about that

Re: [PATCHES] [BUGS] Bug#333854: pg_group file update problems

2005-10-14 Thread Bruce Momjian
I have developed a small patch to 8.0.X that fixes your reported problem: test=> CREATE GROUP g1; CREATE GROUP test=> CREATE USER u1 IN GROUP g1; CREATE USER test=> \! cat /u/pg/data/global/pg_group "g1""u1" test=> CREATE USER u2 IN GR

Re: [PATCHES] [BUGS] Bug#333854: pg_group file update problems

2005-10-14 Thread Tom Lane
Bruce Momjian writes: > In the patch, notice the old comment that suggests we might need to use > CommandCounterIncrement(). ... which you failed to fix in any meaningful way. I'd suggest /* * Advance the commmand counter to ensure we see all results * of current trans

Re: [PATCHES] [BUGS] Bug#333854: pg_group file update problems

2005-10-14 Thread Bruce Momjian
Tom Lane wrote: > Bruce Momjian writes: > > In the patch, notice the old comment that suggests we might need to use > > CommandCounterIncrement(). > > ... which you failed to fix in any meaningful way. I'd suggest > > /* >* Advance the commmand counter to ensure we see all results

Re: [PATCHES] [BUGS] Bug#333854: pg_group file update problems

2005-10-14 Thread Tom Lane
Bruce Momjian writes: > OK, updated patch. I was sort of hoping that you would make the comments agree with the code... regards, tom lane ---(end of broadcast)--- TIP 5: don't forget to increase your free space map settings

Re: [PATCHES] [BUGS] Bug#333854: pg_group file update problems

2005-10-14 Thread Bruce Momjian
Tom Lane wrote: > Bruce Momjian writes: > > OK, updated patch. > > I was sort of hoping that you would make the comments agree with the > code... Oh, you really read those comments? Fixed and attached. -- Bruce Momjian| http://candle.pha.pa.us pgman@candle.pha.pa.

Re: [PATCHES] Documentation typos

2005-10-14 Thread Neil Conway
On Thu, 2005-13-10 at 00:39 -0600, Michael Fuhr wrote: > Correct several typos in the documentation. Are there any remaining objections to this patch? Otherwise, I'll apply it within 24 hours. -Neil ---(end of broadcast)--- TIP 4: Have you search

Re: [PATCHES] Documentation typos

2005-10-14 Thread Michael Fuhr
On Fri, Oct 14, 2005 at 06:18:08PM -0400, Neil Conway wrote: > On Thu, 2005-13-10 at 00:39 -0600, Michael Fuhr wrote: > > Correct several typos in the documentation. > > Are there any remaining objections to this patch? Otherwise, I'll apply > it within 24 hours. I could submit a version with ran