[HACKERS] [PATCH][v8.5] SE-PostgreSQL Patch Updates (r2016)

2009-06-10 Thread KaiGai Kohei
The SE-PostgreSQL patches are updated as follows: 1) http://sepgsql.googlecode.com/files/sepgsql-01-sysatt-8.4beta2-r2016.patch 2) http://sepgsql.googlecode.com/files/sepgsql-02-core-8.4beta2-r2016.patch 3) http://sepgsql.googlecode.com/files/sepgsql-03-writable-8.4beta2-r2016.patch 4) http://sepg

Re: [HACKERS] pgindent run coming

2009-06-10 Thread Tom Lane
Andrew Dunstan writes: > Bruce Momjian wrote: >> I will check on our Postgres shell server right away. > OK, so we got that working, and the consolidated list now contains FBSD > data as well. Um, let's *go* guys. RC1 wrap is scheduled for 18 hours from now. That means it is already too late t

Re: [HACKERS] pgindent run coming

2009-06-10 Thread Andrew Dunstan
Bruce Momjian wrote: The consolidated list comes from Windows(mingw) and Linux. My Cygwin run broke for some reason, and 'objdump --stabs' doesn't seem to do what we need on FBSD, so the output there was empty. If someone knows how to get the typedefs out via objdump on FBSD would they pleas

Re: [HACKERS] Getting error while trying to insert date with the format 'dd-month-yyyy' , 'day-mm-yyyy' etc..

2009-06-10 Thread Tom Lane
Rushabh Lathia writes: > I thought when we are providing the different format into to_char for > datetime then standard datetime input converters should also have the > capability to read that format. to_char is designed to support a near-infinite number of output formats, including many that ar

Re: [HACKERS] Getting error while trying to insert date with the format 'dd-month-yyyy' , 'day-mm-yyyy' etc..

2009-06-10 Thread Rushabh Lathia
On Thu, Jun 11, 2009 at 12:02 AM, Peter Eisentraut wrote: > On Wednesday 10 June 2009 17:10:42 Tom Lane wrote: > > Rushabh Lathia writes: > > > Getting error while trying to insert date with the format > 'dd-month-' > > > , 'day-mm-' (format which add the space in between the date ) etc.

Re: [HACKERS] pgindent run coming

2009-06-10 Thread Bruce Momjian
Andrew Dunstan wrote: > > > Andrew Dunstan wrote: > > > > > > I am doing runs as requested on various platforms to extract the > > typedef lists. Linux is done, Windows (mingw) is running, FBSD and > > Cygwin to come. > > > > Results in a few hours. The buildfarm will have a consolidated list

Re: [HACKERS] pgindent run coming

2009-06-10 Thread Andrew Dunstan
Andrew Dunstan wrote: I am doing runs as requested on various platforms to extract the typedef lists. Linux is done, Windows (mingw) is running, FBSD and Cygwin to come. Results in a few hours. The buildfarm will have a consolidated list. The consolidated list comes from Windows(ming

[HACKERS] 8.4beta[12] set returning function fails -- was O.K. with 8.3

2009-06-10 Thread J. Greg Davidson
Dear PostgreSQL Hackers, Through PostgreSQL 8.3, both of the following functions worked, using generate_series(array_lower($1, 1), array_upper($1, 1)) i instead of generate_subscripts($1, 1). With PostgreSQL 8.4, both are accepted, but only the second one works, regardless of whether I use gene

Re: [HACKERS] Display of foreign keys in psql

2009-06-10 Thread Tom Lane
Greg Stark writes: > On Wed, Jun 10, 2009 at 11:21 PM, Tom Lane wrote: >> Greg Stark writes: >>> TABLE test2 CONSTRAINT test2_y_fkey FOREIGN KEY (y) REFERENCES test1(a); >> >> +1 > Even there the "TABLE" is kind of optional. It would stlil make sense as > Referenced by: > test2 CONSTRAINT test

Re: [HACKERS] Display of foreign keys in psql

2009-06-10 Thread Greg Stark
On Wed, Jun 10, 2009 at 11:21 PM, Tom Lane wrote: > Greg Stark writes: >> So perhaps something like: > >> Referenced by: >> TABLE test2 CONSTRAINT test2_y_fkey FOREIGN KEY (y) REFERENCES test1(a); > > +1 > > ... although making it *really* copy-and-pastable would require a bit > more attention to

Re: [HACKERS] [BUGS] Cursor with hold emits the same row more than once across commits in 8.3.7

2009-06-10 Thread Mark Kirkwood
Tom Lane wrote: Given that RC freeze is nearly upon us for 8.4, and that we need a reasonably non-invasive fix for 8.3 anyway, I propose that for now we just deal with the syncscan issue by tweaking heap_rescan so that rs_startblock doesn't get changed. It looks like that's about a three-line pa

Re: [HACKERS] Display of foreign keys in psql

2009-06-10 Thread Tom Lane
Greg Stark writes: > So perhaps something like: > Referenced by: > TABLE test2 CONSTRAINT test2_y_fkey FOREIGN KEY (y) REFERENCES test1(a); +1 ... although making it *really* copy-and-pastable would require a bit more attention to detail than I bet it's gotten. (Schema qualification and double

Re: [HACKERS] Not quite a security hole in internal_in

2009-06-10 Thread Tom Lane
I wrote: > Sergey Burladyan writes: >> How about contrib/ ? I have this in my test 8.3.7 database: > That stuff should all be marked strict ... on the whole I'm not sure > that contrib is null-safe anyway, independently of this particular > issue. AFAIK no one's really gone through it. So I jus

Re: [HACKERS] Display of foreign keys in psql

2009-06-10 Thread Greg Stark
On Wed, Jun 10, 2009 at 11:04 PM, Tom Lane wrote: > > Or use TABLE: > >  "test2_y_fkey" TABLE test2 FOREIGN KEY (y) REFERENCES test1(a) Hm, one of the things a lot of people said they liked about the existing list is that it was almost copy-pastable as the command to recreate the constraint. If we

Re: [HACKERS] Display of foreign keys in psql

2009-06-10 Thread Brendan Jurd
2009/6/11 Peter Eisentraut : > Referenced by: >  "test2_y_fkey" IN test2 FOREIGN KEY (y) REFERENCES test1(a) > > Is there a magic reason why the IN is capitalized?  (Maybe "from" would be > better anyway?) Isn't "on" the conventional preposition to use here? I would think of this as a foreign key

Re: [HACKERS] Display of foreign keys in psql

2009-06-10 Thread Greg Stark
On Wed, Jun 10, 2009 at 10:58 PM, Bruce Momjian wrote: > >> Is there a magic reason why the IN is capitalized?  (Maybe "from" would be >> better anyway?) > > Probably not.  They were used to capitalizing "IN" for a subquery and it > carried over;   should be lowercase. Well in that line everything

Re: [HACKERS] Display of foreign keys in psql

2009-06-10 Thread Tom Lane
Peter Eisentraut writes: > Referenced by: > "test2_y_fkey" IN test2 FOREIGN KEY (y) REFERENCES test1(a) > Is there a magic reason why the IN is capitalized? (Maybe "from" would be > better anyway?) I think it was probably done to make it more visually distinct from the adjacent identifiers,

Re: [HACKERS] Display of foreign keys in psql

2009-06-10 Thread Kevin Grittner
Bruce Momjian wrote: > Peter Eisentraut wrote: >> Indexes: >> "test1_pkey" PRIMARY KEY, btree (a) >> Referenced by: >> "test2_y_fkey" IN test2 FOREIGN KEY (y) REFERENCES test1(a) >> >> Is there a magic reason why the IN is capitalized? > should be lowercase. What about PRIMARY KEY, FO

Re: [HACKERS] Display of foreign keys in psql

2009-06-10 Thread Bruce Momjian
Peter Eisentraut wrote: > I'm having trouble coming up with a sensible translation for the last line of > this: > > peter=# \d test* > Table "public.test1" > Column | Type | Modifiers > +-+--- > a | integer | not null > Indexes: > "test1_pkey" PRIMARY KE

[HACKERS] Display of foreign keys in psql

2009-06-10 Thread Peter Eisentraut
I'm having trouble coming up with a sensible translation for the last line of this: peter=# \d test* Table "public.test1" Column | Type | Modifiers +-+--- a | integer | not null Indexes: "test1_pkey" PRIMARY KEY, btree (a) Referenced by: "test2_y_fkey"

Re: [HACKERS] pgindent run coming

2009-06-10 Thread Greg Stark
On Wed, Jun 10, 2009 at 9:54 PM, Tom Lane wrote: > Peter Eisentraut writes: I think it usually does that already ... > >> Um, attached you will find a bunch of counterexamples. > > At a quick look, I'm not sure that any of these are in code that hasn't > been edited since the 8.3 pgindent run

Re: [HACKERS] pgindent run coming

2009-06-10 Thread Tom Lane
Peter Eisentraut writes: >>> I think it usually does that already ... > Um, attached you will find a bunch of counterexamples. At a quick look, I'm not sure that any of these are in code that hasn't been edited since the 8.3 pgindent run. regards, tom lane -- Sent via

Re: [HACKERS] pgindent run coming

2009-06-10 Thread Peter Eisentraut
On Wednesday 10 June 2009 22:50:15 Bruce Momjian wrote: > Tom Lane wrote: > > Peter Eisentraut writes: > > > Btw., can you make pgindent remove whitespace at the end of lines? > > > > I think it usually does that already ... > > Yes. Um, attached you will find a bunch of counterexamples. Index:

Re: [HACKERS] pgindent run coming

2009-06-10 Thread Bruce Momjian
Tom Lane wrote: > Greg Stark writes: > > Out of curiosity how different is the output if we don't pass the > > typedef list at all? I'm wondering if the formatting differences are > > things we actually care much about anyways. > > It tends to put extra spaces in variable declarations that are us

Re: [HACKERS] pgindent run coming

2009-06-10 Thread Bruce Momjian
Andrew Dunstan wrote: > > > Bruce Momjian wrote: > > I did a diff, attached, and found some typedefs that don't appear, like > > PortalData. That is defined in our code as: > > > > typedef struct PortalData *Portal; > > > > typedef struct PortalData > > { > > /* Bookkeep

Re: [HACKERS] pgindent run coming

2009-06-10 Thread Bruce Momjian
Tom Lane wrote: > Peter Eisentraut writes: > > Btw., can you make pgindent remove whitespace at the end of lines? > > I think it usually does that already ... Yes. -- Bruce Momjian http://momjian.us EnterpriseDB http://enterprisedb.com + If your life

Re: [HACKERS] pgindent run coming

2009-06-10 Thread Tom Lane
Peter Eisentraut writes: > Btw., can you make pgindent remove whitespace at the end of lines? I think it usually does that already ... regards, tom lane -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://ww

Re: [HACKERS] Multicolumn index corruption on 8.4 beta 2

2009-06-10 Thread Tom Lane
Floris Bos / Maxnet writes: > Hi, > Tom Lane wrote: >> Floris Bos / Maxnet writes: >>> postg...@db:/data$ /opt/postgres/8.4-beta/bin/64/initdb -E SQL_ASCII -X >>> /data/pg_xlog /data/db >>> The database cluster will be initialized with locale en_US.UTF-8. >> >> Oooh, that doesn't look real good

Re: [HACKERS] pgindent run coming

2009-06-10 Thread Peter Eisentraut
On Tuesday 09 June 2009 20:21:35 Bruce Momjian wrote: > It is time to run pgindent on CVS HEAD for 8.4. I am thinking of > running it at zero-hour GMT tomorrow, meaning five hours from now. > Any objections? Btw., can you make pgindent remove whitespace at the end of lines? -- Sent via pgsql-ha

Re: [HACKERS] pgindent run coming

2009-06-10 Thread Tom Lane
Greg Stark writes: > Out of curiosity how different is the output if we don't pass the > typedef list at all? I'm wondering if the formatting differences are > things we actually care much about anyways. It tends to put extra spaces in variable declarations that are using the typedef. Not sure a

Re: [HACKERS] Resolving 8.4 open items

2009-06-10 Thread Tom Lane
Greg Smith writes: > On Wed, 10 Jun 2009, Tom Lane wrote: >> There are also a number of documentation issues open, particularly >> Dimitri's work on documenting the GIST API better, but we can work >> on those later. We've never considered that the RC freeze applies >> to documentation. > I had

Re: [HACKERS] pgindent run coming

2009-06-10 Thread Andrew Dunstan
Bruce Momjian wrote: I did a diff, attached, and found some typedefs that don't appear, like PortalData. That is defined in our code as: typedef struct PortalData *Portal; typedef struct PortalData { /* Bookkeeping data */ ... b

Re: [HACKERS] pgindent run coming

2009-06-10 Thread Greg Stark
Out of curiosity how different is the output if we don't pass the typedef list at all? I'm wondering if the formatting differences are things we actually care much about anyways. -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www

Re: [HACKERS] Resolving 8.4 open items

2009-06-10 Thread Greg Smith
On Wed, 10 Jun 2009, Tom Lane wrote: There are also a number of documentation issues open, particularly Dimitri's work on documenting the GIST API better, but we can work on those later. We've never considered that the RC freeze applies to documentation. I had a question in this area actually

Re: [HACKERS] pgindent run coming

2009-06-10 Thread Bruce Momjian
Andrew Dunstan wrote: > > > Tom Lane wrote: > > Bruce Momjian writes: > > > >> I am unclear why struct pointers are not being formatted properly in > >> function headers but will research it. > >> > > > > Yeah, if we can fix that directly without adding the names to the > > typedef list,

Re: [HACKERS] pgindent run coming

2009-06-10 Thread Andrew Dunstan
Tom Lane wrote: Bruce Momjian writes: I am unclear why struct pointers are not being formatted properly in function headers but will research it. Yeah, if we can fix that directly without adding the names to the typedef list, it would be better. But not something to do right now.

Re: [HACKERS] pgindent run coming

2009-06-10 Thread Tom Lane
Bruce Momjian writes: >> Have you started the pgindent run yet? I have a patch ready for >> the cursor stability issue, but will hold off committing if it might >> create a merge problem for you. > I am waiting for Andrew to tell me he is ready with updated lists for > his platforms. His CGI ou

Re: [HACKERS] pgindent run coming

2009-06-10 Thread Bruce Momjian
Tom Lane wrote: > Bruce Momjian writes: > > I am unclear why struct pointers are not being formatted properly in > > function headers but will research it. > > Yeah, if we can fix that directly without adding the names to the > typedef list, it would be better. But not something to do right now.

Re: [HACKERS] Resolving 8.4 open items

2009-06-10 Thread Tom Lane
Josh Berkus writes: > Tom, >> I suspect there aren't many. What I'm more concerned about is that >> people may have copied the bogus logic for use in their own datatypes. >> (Which is exactly how Matthew Wakeling came to find out the problem.) >> But in any case, this train is leaving the station

Re: [HACKERS] Resolving 8.4 open items

2009-06-10 Thread Josh Berkus
Tom, I suspect there aren't many. What I'm more concerned about is that people may have copied the bogus logic for use in their own datatypes. (Which is exactly how Matthew Wakeling came to find out the problem.) But in any case, this train is leaving the station. Can we put a warning in the

Re: [HACKERS] Resolving 8.4 open items

2009-06-10 Thread Tom Lane
Josh Berkus writes: > If this issue has existed for several versions, and we're not getting a > lot of complaints, it says that either not many people are using cube > and seg or they don't notice performance issues. > Mind you, if performance is terrible, then not many people *would* use > cu

Re: [HACKERS] Getting error while trying to insert date with the format 'dd-month-yyyy' , 'day-mm-yyyy' etc..

2009-06-10 Thread Peter Eisentraut
On Wednesday 10 June 2009 17:10:42 Tom Lane wrote: > Rushabh Lathia writes: > > Getting error while trying to insert date with the format 'dd-month-' > > , 'day-mm-' (format which add the space in between the date ) etc.. > > 1. Why are you bothering with the completely pointless to_char c

Re: [HACKERS] pgindent run coming

2009-06-10 Thread Tom Lane
Bruce Momjian writes: > I am unclear why struct pointers are not being formatted properly in > function headers but will research it. Yeah, if we can fix that directly without adding the names to the typedef list, it would be better. But not something to do right now. Have you started the pgind

Re: [HACKERS] Resolving 8.4 open items

2009-06-10 Thread Josh Berkus
Tom, Alternatively, we can postpone 8.4 till Oleg and Teodor have some spare cycles to look at the patch, but who knows when that will be. Not soon. So, +1 to go ahead. If this issue has existed for several versions, and we're not getting a lot of complaints, it says that either not many pe

Re: [HACKERS] Multicolumn index corruption on 8.4 beta 2

2009-06-10 Thread Josh Berkus
Floris, One more question set: what version of OpenSolaris, and what filesystem are you using? Does the OS have any non-default tuning settings? How did you install or compile PostgreSQL? -- Josh Berkus PostgreSQL Experts Inc. www.pgexperts.com -- Sent via pgsql-hackers mailing list (pgsq

Re: [HACKERS] pgindent run coming

2009-06-10 Thread Bruce Momjian
Tom Lane wrote: > Bruce Momjian writes: > > OK, I have found the cause of the script error, and it was my fault. A > > month after we ran pgindent for 8.3 (December 2007), I received this > > issue from Tom: > > > http://archives.postgresql.org/pgsql-hackers/2007-12/msg00800.php > >> Something I

Re: [HACKERS] Resolving 8.4 open items

2009-06-10 Thread Tom Lane
Josh Berkus writes: > On 6/10/09 10:40 AM, Tom Lane wrote: >> * contrib/seg and contrib/cube GiST index support have performance issues > Wasn't there an issue that fixing this requires a data format change? Well, that means it probably doesn't get fixed till 8.5. I'm annoyed at that, but such

Re: [HACKERS] Multicolumn index corruption on 8.4 beta 2

2009-06-10 Thread Andy Colson
Josh Berkus wrote: Andy, 6) To the -hackers: I write the records and then refind them in the exact same order, would it be a better test to search for records in a more random order? would it make a difference? Would searching for some but not all make a difference? Are you on OpenSolaris? C

Re: [HACKERS] Problem with listen_addresses = '*' on 8.4beta2 on AIX

2009-06-10 Thread Andrew Dunstan
Merlin Moncure wrote: On Wed, Jun 10, 2009 at 12:13 PM, Tom Lane wrote: "Albe Laurenz" writes: Maybe it would be good enough to state that AIX is only supported with certain fixpacks installed? And do without all the AIX hacks? That would definitely be a good-enough solution

Re: [HACKERS] Problem with listen_addresses = '*' on 8.4beta2 on AIX

2009-06-10 Thread Merlin Moncure
On Wed, Jun 10, 2009 at 12:13 PM, Tom Lane wrote: > "Albe Laurenz" writes: >> Maybe it would be good enough to state that AIX is only supported with >> certain fixpacks installed? And do without all the AIX hacks? > > That would definitely be a good-enough solution in my opinion, > as long as we c

Re: [HACKERS] Resolving 8.4 open items

2009-06-10 Thread Josh Berkus
On 6/10/09 10:40 AM, Tom Lane wrote: * contrib/seg and contrib/cube GiST index support have performance issues Wasn't there an issue that fixing this requires a data format change? -- Josh Berkus PostgreSQL Experts Inc. www.pgexperts.com -- Sent via pgsql-hackers mailing list (pgsql-hackers@p

Re: [HACKERS] Resolving 8.4 open items

2009-06-10 Thread Tom Lane
Magnus Hagander writes: > Tom Lane wrote: >> * do we need to worry about re-encoding file path names? > Is there really something new here for 8.4? Haven't we lived with this > same thing previously? Right, it's a pre-existing issue --- any misbehavior in this area goes back to the beginning.

Re: [HACKERS] Multicolumn index corruption on 8.4 beta 2

2009-06-10 Thread Josh Berkus
Andy, 6) To the -hackers: I write the records and then refind them in the exact same order, would it be a better test to search for records in a more random order? would it make a difference? Would searching for some but not all make a difference? Are you on OpenSolaris? Can you give your scr

Re: [HACKERS] Resolving 8.4 open items

2009-06-10 Thread Magnus Hagander
Tom Lane wrote: > If we are to make the proposed schedule for 8.4 (ie, wrap RC1 tomorrow) > we've got to get pretty hard-nosed about closing out the open items at > http://wiki.postgresql.org/wiki/PostgreSQL_8.4_Open_Items > > The current list is: > > * do we need to worry about re-encoding file

Re: [HACKERS] PlannerInfo Structure

2009-06-10 Thread Tom Lane
Ana Carolina Brito de Almeida writes: > I created a function which needs build or find a relation (RelOptInfo). > The build is ok. > On the other hand, I can't find the relation. I have all informations that > pg_plan_query has: querytree (Query), cursorOptions (int), > boundParams(ParamListInfo)

[HACKERS] Resolving 8.4 open items

2009-06-10 Thread Tom Lane
If we are to make the proposed schedule for 8.4 (ie, wrap RC1 tomorrow) we've got to get pretty hard-nosed about closing out the open items at http://wiki.postgresql.org/wiki/PostgreSQL_8.4_Open_Items The current list is: * do we need to worry about re-encoding file path names? There is perhaps

Re: [HACKERS] Plan time Improvement - 64bit bitmapset

2009-06-10 Thread Andres Freund
Hi, On 06/10/2009 06:01 PM, Kevin Grittner wrote: Andres Freund wrote: - Sometimes adding a single join more/less dropped the planning time to a fraction - strange. - The same with changing {join,from}_collapse_limit - sometimes changing it yields plan times different by orders of magnitudes i

[HACKERS] PlannerInfo Structure

2009-06-10 Thread Ana Carolina Brito de Almeida
Hi all, I created a function which needs build or find a relation (RelOptInfo). The build is ok. On the other hand, I can't find the relation. I have all informations that pg_plan_query has: querytree (Query), cursorOptions (int), boundParams(ParamListInfo) and plan (PlannedStmt). So, how can I ta

Re: [HACKERS] [BUGS] Cursor with hold emits the same row more than once across commits in 8.3.7

2009-06-10 Thread Tom Lane
Given that RC freeze is nearly upon us for 8.4, and that we need a reasonably non-invasive fix for 8.3 anyway, I propose that for now we just deal with the syncscan issue by tweaking heap_rescan so that rs_startblock doesn't get changed. It looks like that's about a three-line patch. The question

Re: [HACKERS] pgindent run coming

2009-06-10 Thread Tom Lane
Bruce Momjian writes: > OK, I have found the cause of the script error, and it was my fault. A > month after we ran pgindent for 8.3 (December 2007), I received this > issue from Tom: > http://archives.postgresql.org/pgsql-hackers/2007-12/msg00800.php >> Something I noticed the other day is that

Re: [HACKERS] Problem with listen_addresses = '*' on 8.4beta2 on AIX

2009-06-10 Thread Tom Lane
"Albe Laurenz" writes: > Maybe it would be good enough to state that AIX is only supported with > certain fixpacks installed? And do without all the AIX hacks? That would definitely be a good-enough solution in my opinion, as long as we can document which fixes are needed for which releases.

Re: [HACKERS] pgindent run coming

2009-06-10 Thread Bruce Momjian
Andrew Dunstan wrote: > > > Bruce Momjian wrote: > > > > Good point. Here is another diff I need you to make to the pl file. > > > > Done. Linux run under way. > > > If you want to make your pl file the official version and replace the > > shell script in CVS, that is fine with me. Do you

Re: [HACKERS] Plan time Improvement - 64bit bitmapset

2009-06-10 Thread Kevin Grittner
Andres Freund wrote: > - Sometimes adding a single join more/less dropped the planning time > to a fraction - strange. > - The same with changing {join,from}_collapse_limit - sometimes > changing it yields plan times different by orders of magnitudes in > both directions. That seems like the p

Re: [HACKERS] pgindent run coming

2009-06-10 Thread Bruce Momjian
Bruce Momjian wrote: > Tom Lane wrote: > > Bruce Momjian writes: > > > I saw a few odd things. Most importantly, it seems 'stat' was > > > introduced as a typedef on _both_ lists, yielding weird changes like: > > > > The standard headers do define "struct stat". I wonder whether the > > objdump

Re: [HACKERS] Problem with listen_addresses = '*' on 8.4beta2 on AIX

2009-06-10 Thread Albe Laurenz
Merlin Moncure wrote: > I wrote: >> I can try, but I only have AIX 5.3 to test with, so I won't be able >> to test for bugs in other versions. >> >> I would need to know what the problems with getaddrinfo on AIX are. >> >> I'll also try to investigate known bugs in AIX' getaddrinfo. >> >> AIX 4.3 h

Re: [HACKERS] pgindent run coming

2009-06-10 Thread Andrew Dunstan
Bruce Momjian wrote: Good point. Here is another diff I need you to make to the pl file. Done. Linux run under way. If you want to make your pl file the official version and replace the shell script in CVS, that is fine with me. Do you want me to do that? It needs to be done in

Re: [HACKERS] pgindent run coming

2009-06-10 Thread Bruce Momjian
Andrew Dunstan wrote: > Well, sometimes I build it and they don't come ;-). > > I don't have every platform under the sun that I can run this on, > although I do now have an FBSD VM that I didn't have when I worked on > this previously. If you're actually going to use it I'll set it up as a > b

Re: [HACKERS] pgindent run coming

2009-06-10 Thread Andrew Dunstan
Bruce Momjian wrote: Andrew Dunstan wrote: Bruce Momjian wrote: OK, Andrew, would you use the find_typedef file that is in CVS HEAD and run that. I think that will fix our problem and then I can use the buildfarm version. How often does that run and does it pull the script from CVS

Re: [HACKERS] pgindent run coming

2009-06-10 Thread Tom Lane
Bruce Momjian writes: > Tom Lane wrote: >> We don't have a lot of time for research. Maybe the best thing is to >> just manually remove stat from the typedef list (along with anything >> else that clearly shouldn't be there)? > I agree we are running out of time so I will be running pgindent in

Re: [HACKERS] pgindent run coming

2009-06-10 Thread Bruce Momjian
Tom Lane wrote: > Bruce Momjian writes: > > I saw a few odd things. Most importantly, it seems 'stat' was > > introduced as a typedef on _both_ lists, yielding weird changes like: > > The standard headers do define "struct stat". I wonder whether the > objdump kluge we are using is unable to di

Re: [HACKERS] Multicolumn index corruption on 8.4 beta 2

2009-06-10 Thread Andy Colson
Floris Bos / Maxnet wrote: The following settings differ from the defaults: -- shared_buffers=3500MB maintenance_work_mem = 128MB fsync = off synchronous_commit = off checkpoint_segments = 25 -- == Table layout == -- Table "public.posts_index" Column |

Re: [HACKERS] Getting error while trying to insert date with the format 'dd-month-yyyy' , 'day-mm-yyyy' etc..

2009-06-10 Thread Tom Lane
Rushabh Lathia writes: > Getting error while trying to insert date with the format 'dd-month-' , > 'day-mm-' (format which add the space in between the date ) etc.. 1. Why are you bothering with the completely pointless to_char call at all? 2. It is not a bug that to_char is capable of e

Re: [HACKERS] pgindent run coming

2009-06-10 Thread Tom Lane
Simon Riggs writes: > On Tue, 2009-06-09 at 13:21 -0400, Bruce Momjian wrote: >> It is time to run pgindent on CVS HEAD for 8.4. I am thinking of >> running it at zero-hour GMT tomorrow, meaning five hours from now. > Why don't we do this automatically after each individual commit? It's not ve

Re: [HACKERS] pgindent run coming

2009-06-10 Thread Bruce Momjian
Andrew Dunstan wrote: > > > Bruce Momjian wrote: > > OK, Andrew, would you use the find_typedef file that is in CVS HEAD and > > run that. I think that will fix our problem and then I can use the > > buildfarm version. How often does that run and does it pull the script > > from CVS HEAD? > >

Re: [HACKERS] page is uninitialized --- fixing

2009-06-10 Thread Simon Riggs
On Tue, 2009-06-09 at 17:28 -0400, Tom Lane wrote: > Is there any particular reason to suppose that the empty pages appeared > during a crash recovery? > > Have you read through md.c? I seem to recall there are some slightly > squirrelly choices made there about segment-extension behavior. May

Re: [HACKERS] Problem with listen_addresses = '*' on 8.4beta2 on AIX

2009-06-10 Thread Merlin Moncure
On Wed, Jun 10, 2009 at 3:38 AM, Albe Laurenz wrote: > Tom Lane wrote: >> Drat.  Okay, that easy solution probably won't fly.  Anyone with AIX >> access want to try to develop a patch that covers this case without >> breaking the other ones? > > I can try, but I only have AIX 5.3 to test with, so I

[HACKERS] [PATCH v2] [libpq] Try to avoid manually masking SIGPIPEs on every send()

2009-06-10 Thread Jeremy Kerr
Currently, libpq will wrap each send() call on the connection with two system calls to mask SIGPIPEs. This results in 3 syscalls instead of one, and (on Linux) can lead to high contention on the signal mask locks in threaded apps. We have a couple of other methods to avoid SIGPIPEs: sockopt(SO_NOS

Re: [HACKERS] [PATCH 2/2] [libpq] Try to avoid manually masking SIGPIPEs on every send()

2009-06-10 Thread Jeremy Kerr
Marko, > optval seems used without initialization. Dang, I was checking for the SO_NOSIGPIPE flag, and didn't check for optval. New patch coming... Cheers, Jeremy -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresq

Re: [HACKERS] Plan time Improvement - 64bit bitmapset

2009-06-10 Thread Andres Freund
Hi, On 06/10/2009 01:38 PM, Gregory Stark wrote: Andres Freund writes: Plan time (averaged) without change: cnt: 40 (4 times per session) avg: 4572ms Plan time (averaged) with change: cnt: 40 (4 times per session) avg: 4236ms ~7% difference. Same with higher number of repetitions and with m

Re: [HACKERS] Plan time Improvement - 64bit bitmapset

2009-06-10 Thread Gregory Stark
Andres Freund writes: > Plan time (averaged) without change: > cnt: 40 (4 times per session) > avg: 4572ms > > Plan time (averaged) with change: > cnt: 40 (4 times per session) > avg: 4236ms > > ~7% difference. Same with higher number of repetitions and with most other > planner settings I tried

[HACKERS] Re: [PATCH 2/2] [libpq] Try to avoid manually masking SIGPIPEs on every send()

2009-06-10 Thread Marko Kreen
On 6/10/09, Jeremy Kerr wrote: > + int optval; > + if (!setsockopt(conn->sock, > SOL_SOCKET, SO_NOSIGPIPE, > + (char *)&optval, > sizeof(optval))) optval seems u

Re: [HACKERS] Plan time Improvement - 64bit bitmapset

2009-06-10 Thread Andres Freund
Hi, On 06/03/2009 06:42 PM, Tom Lane wrote: Andres Freund writes: On 06/03/2009 06:21 PM, Tom Lane wrote: I find this *really* hard to believe, because I've never seen the bitmap support operations show up noticeably at all in profiles. What sort of queries are you testing? Many left joins

[HACKERS] Getting error while trying to insert date with the format 'dd-month-yyyy' , 'day-mm-yyyy' etc..

2009-06-10 Thread Rushabh Lathia
Getting error while trying to insert date with the format 'dd-month-' , 'day-mm-' (format which add the space in between the date ) etc.. Testcase: postgres=# \d t Table "public.t" Column | Type | Modifiers +--+--- a | date | postgres=# insert into

Re: [HACKERS] Problem with listen_addresses = '*' on 8.4beta2 on AIX

2009-06-10 Thread Albe Laurenz
Tom Lane wrote: > Drat. Okay, that easy solution probably won't fly. Anyone with AIX > access want to try to develop a patch that covers this case without > breaking the other ones? I can try, but I only have AIX 5.3 to test with, so I won't be able to test for bugs in other versions. I would n