Re: [HACKERS] Typing Records

2010-08-24 Thread Joe Conway
On 08/23/2010 08:33 PM, David E. Wheeler wrote: I've been trying to come up with a simpler way to iterate over a series of values in pgTAP tests than by creating a table, inserting rows, and then selecting from the table. The best I've come up with so far is: snip Aside from that, might

Re: [HACKERS] patch (for 9.1) string functions

2010-08-24 Thread Itagaki Takahiro
we can hold this function most simple. -- Itagaki Takahiro stringfunc-20100824.diff Description: Binary data -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers

Re: [HACKERS] patch (for 9.1) string functions

2010-08-24 Thread Pavel Stehule
2010/8/24 Itagaki Takahiro itagaki.takah...@gmail.com: I applied the attached patch to HEAD. concat(), concat_ws(), left(), right(), and reverse() are in it, but format() and sprintf() are not. It's my understanding that we don't have consensus about the best syntax for the formatting

Re: [Glue] [HACKERS] Deadlock bug

2010-08-24 Thread Kevin Grittner
Josh Berkus wrote: the behavior was the same up to the second UPDATE on Process 2, at which point there was no deadlock. Process 2 was able to commit, at which point Process 1 failed with: ERROR: could not serialize access due to concurrent update Does this happen immediately, not

Re: Interruptible sleeps (was Re: [HACKERS] CommitFest 2009-07: Yay, Kevin! Thanks, reviewers!)

2010-08-24 Thread Heikki Linnakangas
On 24/08/10 04:08, Alvaro Herrera wrote: Excerpts from Tom Lane's message of lun ago 23 19:44:02 -0400 2010: Heikki Linnakangasheikki.linnakan...@enterprisedb.com writes: [ latch proposal ] This seems reasonably clean as far as signal conditions generated internally to Postgres go, but I

Re: [HACKERS] INSERT and parentheses

2010-08-24 Thread Marko Tiikkaja
On 2010-08-24 8:25 AM +0300, igor polishchuk wrote: Marko et al, This is my first ever attempt of a patch review just for learning the procedure. I'm not a postgres developer, so the review is partial and mostly from the usability prospective. That's all right. I'm sure any help is

Re: [HACKERS] WIP: extensible enums

2010-08-24 Thread Andrew Dunstan
On 08/23/2010 07:12 PM, Bruce Momjian wrote: Josh Berkus wrote: On 8/23/10 12:20 PM, Tom Lane wrote: Josh Berkusj...@agliodbs.com writes: I really don't see the value in making a command substantially less intuitive in order to avoid a single keyword, unless it affects areas of Postgres

Re: [HACKERS] patch (for 9.1) string functions

2010-08-24 Thread Erik Rijkers
On Tue, August 24, 2010 08:32, Itagaki Takahiro wrote: I applied the attached patch to HEAD. concat(), concat_ws(), left(), right(), and reverse() are in it, but format() and sprintf() are not. +1 to add also sprintf Erik Rijkers -- Sent via pgsql-hackers mailing list

Re: [HACKERS] Fw: patch for pg_ctl.c to add windows service start-type

2010-08-24 Thread Quan Zongliang
Which files need be modified? sgml, html, and man-page or only sgml? I am not familiar with this. On Sun, 22 Aug 2010 08:25:59 -0700 David Fetter da...@fetter.org wrote: On Sun, Aug 22, 2010 at 10:03:32PM +0800, Quan Zongliang wrote: Sure, I agree. New patch attached. How about this?

Re: [HACKERS] WIP: extensible enums

2010-08-24 Thread Bruce Momjian
Andrew Dunstan wrote: On 08/23/2010 07:12 PM, Bruce Momjian wrote: Josh Berkus wrote: On 8/23/10 12:20 PM, Tom Lane wrote: Josh Berkusj...@agliodbs.com writes: I really don't see the value in making a command substantially less intuitive in order to avoid a single keyword, unless

Re: [HACKERS] Fw: patch for pg_ctl.c to add windows service start-type

2010-08-24 Thread Magnus Hagander
On Tue, Aug 24, 2010 at 2:05 PM, Quan Zongliang quanzongli...@gmail.com wrote: Which files need be modified? sgml, html, and man-page or only sgml? I am not familiar with this. Only SGML. HTML and man pages are generated from the SGML. --  Magnus Hagander  Me: http://www.hagander.net/  

Re: [HACKERS] [BUGS] BUG #5305: Postgres service stops when closing Windows session

2010-08-24 Thread Bruce Momjian
Robert Haas wrote: [moving to -hackers] On Thu, Aug 19, 2010 at 9:43 PM, Robert Haas robertmh...@gmail.com wrote: I suspect this is the same problem as bug #4897, and probably also the same problem as this: http://archives.postgresql.org/pgsql-bugs/2009-08/msg00114.php and maybe also

Re: [HACKERS] Return of the Solaris vacuum polling problem -- anyone remember this?

2010-08-24 Thread Bruce Momjian
Applied. --- Bruce Momjian wrote: Tom Lane wrote: Bruce Momjian br...@momjian.us writes: OK, I have attached a proposed patch to improve this. I moved the pg_clog mention to a new paragraph and linked it to the

Re: [HACKERS] [BUGS] BUG #5305: Postgres service stops when closing Windows session

2010-08-24 Thread Robert Haas
On Tue, Aug 24, 2010 at 8:57 AM, Bruce Momjian br...@momjian.us wrote: Robert Haas wrote: [moving to -hackers] On Thu, Aug 19, 2010 at 9:43 PM, Robert Haas robertmh...@gmail.com wrote: I suspect this is the same problem as bug #4897, and probably also the same problem as this:

Re: [HACKERS] [BUGS] BUG #5305: Postgres service stops when closing Windows session

2010-08-24 Thread Bruce Momjian
Robert Haas wrote: This isn't really a fix for the bug in the sense that the nicest thing of all would be to prevent the child from exiting abnormally in the first place. ?But it's far from clear that we can control that. This URL has some interesting details on our problem: ? ? ?

Re: [HACKERS] [BUGS] BUG #5305: Postgres service stops when closing Windows session

2010-08-24 Thread Tom Lane
Bruce Momjian br...@momjian.us writes: Robert Haas wrote: Yeah, that seems very plausible, although exactly how to verify I don't know. And here is confirmation from the Microsoft web site: In some instances, calling GetExitCode() against the failed process indicates the

Re: [HACKERS] [BUGS] BUG #5305: Postgres service stops when closing Windows session

2010-08-24 Thread Bruce Momjian
Tom Lane wrote: Bruce Momjian br...@momjian.us writes: Robert Haas wrote: Yeah, that seems very plausible, although exactly how to verify I don't know. And here is confirmation from the Microsoft web site: In some instances, calling GetExitCode() against the failed

Re: [HACKERS] [BUGS] BUG #5305: Postgres service stops when closing Windows session

2010-08-24 Thread Bruce Momjian
Tom Lane wrote: Bruce Momjian br...@momjian.us writes: Robert Haas wrote: Yeah, that seems very plausible, although exactly how to verify I don't know. And here is confirmation from the Microsoft web site: In some instances, calling GetExitCode() against the failed

Re: [HACKERS] Typing Records

2010-08-24 Thread Tom Lane
David E. Wheeler da...@kineticode.com writes: I've been trying to come up with a simpler way to iterate over a series of values in pgTAP tests than by creating a table, inserting rows, and then selecting from the table. The best I've come up with so far is: CREATE TYPE vcmp AS ( lv

[HACKERS] EXPLAIN doesn't show the actual function expression for FunctionScan

2010-08-24 Thread Tom Lane
So I got annoyed by $SUBJECT just now while chasing Wheeler's bug report. Seems like this would be a good thing to print. Should it appear always, or just with VERBOSE, or ??? regards, tom lane -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To

Re: [HACKERS] EXPLAIN doesn't show the actual function expression for FunctionScan

2010-08-24 Thread David Fetter
On Tue, Aug 24, 2010 at 10:40:30AM -0400, Tom Lane wrote: So I got annoyed by $SUBJECT just now while chasing Wheeler's bug report. Seems like this would be a good thing to print. Should it appear always, or just with VERBOSE, or ??? +1 for always. Cheers, David. -- David Fetter

Re: [HACKERS] EXPLAIN doesn't show the actual function expression for FunctionScan

2010-08-24 Thread Robert Haas
On Tue, Aug 24, 2010 at 10:40 AM, Tom Lane t...@sss.pgh.pa.us wrote: So I got annoyed by $SUBJECT just now while chasing Wheeler's bug report.  Seems like this would be a good thing to print.  Should it appear always, or just with VERBOSE, or ??? I think showing it always is reasonable. I'd

Re: [HACKERS] EXPLAIN doesn't show the actual function expression for FunctionScan

2010-08-24 Thread Robert Haas
On Tue, Aug 24, 2010 at 10:56 AM, Robert Haas robertmh...@gmail.com wrote: On Tue, Aug 24, 2010 at 10:40 AM, Tom Lane t...@sss.pgh.pa.us wrote: So I got annoyed by $SUBJECT just now while chasing Wheeler's bug report.  Seems like this would be a good thing to print.  Should it appear always,

Re: [HACKERS] EXPLAIN doesn't show the actual function expression for FunctionScan

2010-08-24 Thread Tom Lane
Robert Haas robertmh...@gmail.com writes: On Tue, Aug 24, 2010 at 10:56 AM, Robert Haas robertmh...@gmail.com wrote: I think showing it always is reasonable.  I'd like to see it printed in a form such that casting to regproc will succeed. On second thought, that second sentence may not make

Re: [HACKERS] Typing Records

2010-08-24 Thread David E. Wheeler
On Aug 23, 2010, at 11:24 PM, Joe Conway wrote: Maybe something like this? select cmp_ok(a,b,c) from ( values('1.2.2'::varchar, '='::text, '1.2.2'::varchar), ('1.2.23', '=', '1.2.23'), ('1.2.42', '=', '1.2.32') ) as ss(a, b, c); cmp_ok t t f (3 rows)

Re: [HACKERS] Typing Records

2010-08-24 Thread David E. Wheeler
On Aug 24, 2010, at 7:05 AM, Tom Lane wrote: You could do it like this: SELECT cmp_ok(lv, op, rv) FROM unnest(ARRAY[ ROW('1.2.2', '=', '1.2.2'), ROW('1.2.23', '=', '1.2.23') ]::vcmp[]); Oh, duh. :-) psql:t/types.pg:205: ERROR: invalid memory alloc request size

Re: [HACKERS] EXPLAIN doesn't show the actual function expression for FunctionScan

2010-08-24 Thread Robert Haas
On Tue, Aug 24, 2010 at 11:06 AM, Tom Lane t...@sss.pgh.pa.us wrote: Robert Haas robertmh...@gmail.com writes: On Tue, Aug 24, 2010 at 10:56 AM, Robert Haas robertmh...@gmail.com wrote: I think showing it always is reasonable.  I'd like to see it printed in a form such that casting to regproc

Re: [HACKERS] EXPLAIN doesn't show the actual function expression for FunctionScan

2010-08-24 Thread Andres Freund
Hi, On Tuesday 24 August 2010 16:40:30 Tom Lane wrote: So I got annoyed by $SUBJECT just now while chasing Wheeler's bug report. Seems like this would be a good thing to print. Should it appear always, or just with VERBOSE, or ??? I vote for only showing it with verbose - not that its a new

Re: [HACKERS] Typing Records

2010-08-24 Thread Tom Lane
David E. Wheeler da...@kineticode.com writes: On Aug 24, 2010, at 7:05 AM, Tom Lane wrote: I get a core dump on that one ... looking ... Well I'm glad I reported it, then. The issue seems to be that given a construct like ARRAY[ ROW('1.2.2'::semver, '='::text,

Re: [HACKERS] EXPLAIN doesn't show the actual function expression for FunctionScan

2010-08-24 Thread Tom Lane
Robert Haas robertmh...@gmail.com writes: If you try to put all that on the same line, I think it might get awkwardly long. Perhaps something like: Function Scan on function_name Expression: function_name(function_arg1, function_arg2, ...) Yeah, that's what I had in mind, but I'm still

Re: [HACKERS] EXPLAIN doesn't show the actual function expression for FunctionScan

2010-08-24 Thread Tom Lane
Andres Freund and...@anarazel.de writes: On Tuesday 24 August 2010 16:40:30 Tom Lane wrote: So I got annoyed by $SUBJECT just now while chasing Wheeler's bug report. Seems like this would be a good thing to print. Should it appear always, or just with VERBOSE, or ??? I vote for only

Re: [HACKERS] EXPLAIN doesn't show the actual function expression for FunctionScan

2010-08-24 Thread Andres Freund
On Tuesday 24 August 2010 17:36:50 Tom Lane wrote: Andres Freund and...@anarazel.de writes: On Tuesday 24 August 2010 16:40:30 Tom Lane wrote: So I got annoyed by $SUBJECT just now while chasing Wheeler's bug report. Seems like this would be a good thing to print. Should it appear

Re: [HACKERS] Typing Records

2010-08-24 Thread Robert Haas
On Tue, Aug 24, 2010 at 11:32 AM, Tom Lane t...@sss.pgh.pa.us wrote: I think it wouldn't take too much code to defend against this in transformArrayExpr, but I'm a tad worried about whether there are similar cases elsewhere.  The generic problem is that we suppose that different values are

Re: [HACKERS] EXPLAIN doesn't show the actual function expression for FunctionScan

2010-08-24 Thread Alvaro Herrera
Excerpts from Tom Lane's message of mar ago 24 11:36:50 -0400 2010: Andres Freund and...@anarazel.de writes: On Tuesday 24 August 2010 16:40:30 Tom Lane wrote: So I got annoyed by $SUBJECT just now while chasing Wheeler's bug report. Seems like this would be a good thing to print. Should

Re: [HACKERS] EXPLAIN doesn't show the actual function expression for FunctionScan

2010-08-24 Thread Tom Lane
I wrote: Robert Haas robertmh...@gmail.com writes: If you try to put all that on the same line, I think it might get awkwardly long. Perhaps something like: Function Scan on function_name Expression: function_name(function_arg1, function_arg2, ...) Yeah, that's what I had in mind, but I'm

Re: [HACKERS] Typing Records

2010-08-24 Thread Tom Lane
Robert Haas robertmh...@gmail.com writes: On Tue, Aug 24, 2010 at 11:32 AM, Tom Lane t...@sss.pgh.pa.us wrote: I think it wouldn't take too much code to defend against this in transformArrayExpr, but I'm a tad worried about whether there are similar cases elsewhere.  The generic problem is

Re: [HACKERS] Problem Using PQcancel in a Synchronous Query

2010-08-24 Thread Eric Simon
Tom, That was it! My implementation of cancel() was swallowing the result message. Thanks so much, I've got it working now. -- Eric Simon The IQ Group, Inc. -Original Message- From: pgsql-hackers-ow...@postgresql.org [mailto:pgsql-hackers-ow...@postgresql.org] On Behalf Of Tom Lane

Re: [HACKERS] [ADMIN] Unable to drop role

2010-08-24 Thread Alvaro Herrera
Excerpts from Tom Lane's message of mar ago 24 09:36:05 -0400 2010: McGehee, Robert robert.mcge...@geodecapital.com writes: Thanks Tom and Alvaro for clearing up my confusion. \l showed that a485099 had both (C)reate and (T)emporary access. Revoking those allowed me to drop the role. Thanks

Re: [HACKERS] EXPLAIN doesn't show the actual function expression for FunctionScan

2010-08-24 Thread David Fetter
On Tue, Aug 24, 2010 at 12:06:34PM -0400, Tom Lane wrote: I wrote: Robert Haas robertmh...@gmail.com writes: If you try to put all that on the same line, I think it might get awkwardly long. Perhaps something like: Function Scan on function_name Expression:

[HACKERS] About debug two versions of postgresql in eclipse

2010-08-24 Thread Pei He
Hi, I want to run two different versions of postgresql-8.2.5 under eclipse. But, it requires me to change PGDATA and LD_LIBRARY_PATH to switch. Moreover, to let eclipse know the changes, I need to modify .profile under my home folder, and log out and log in. (I am using Ubuntu.) Is there a way

Re: [HACKERS] EXPLAIN doesn't show the actual function expression for FunctionScan

2010-08-24 Thread Greg Stark
On Tue, Aug 24, 2010 at 5:06 PM, Tom Lane t...@sss.pgh.pa.us wrote:  Function Scan on pg_catalog.unnest f  (cost=0.00..1.50 rows=100 width=96)   Output: (((lv)::text || op) || (rv)::text)   Function Call: unnest(ARRAY[ROW(('1.2.2'::text)::semver, '='::text, ('1.2.2'::text)::semver),

Re: [HACKERS] [ADMIN] Unable to drop role

2010-08-24 Thread Tom Lane
Alvaro Herrera alvhe...@commandprompt.com writes: Happy to change it to whatever is deemed appropriate. privileges for %s sounds good; I'll do that unless somebody comes up with a better idea which outvotes this one. Backpatch all the way to 8.1? Code doesn't exist prior to that. I'd vote

Re: [HACKERS] EXPLAIN doesn't show the actual function expression for FunctionScan

2010-08-24 Thread Tom Lane
Greg Stark gsst...@mit.edu writes: On Tue, Aug 24, 2010 at 5:06 PM, Tom Lane t...@sss.pgh.pa.us wrote:  Function Scan on pg_catalog.unnest f  (cost=0.00..1.50 rows=100 width=96)   Output: (((lv)::text || op) || (rv)::text)   Function Call: unnest(ARRAY[ROW(('1.2.2'::text)::semver, '='::text,

Re: [HACKERS] Typing Records

2010-08-24 Thread Robert Haas
On Tue, Aug 24, 2010 at 12:27 PM, Tom Lane t...@sss.pgh.pa.us wrote: This is a crash in released branches, so we have to have a back-patchable fix.  Anything that gets out from under the typmod issue isn't going to be back-patchable. I nominate that comment for understatement of the year. --

Re: [HACKERS] EXPLAIN doesn't show the actual function expression for FunctionScan

2010-08-24 Thread Robert Haas
On Tue, Aug 24, 2010 at 1:21 PM, Tom Lane t...@sss.pgh.pa.us wrote: Greg Stark gsst...@mit.edu writes: On Tue, Aug 24, 2010 at 5:06 PM, Tom Lane t...@sss.pgh.pa.us wrote:  Function Scan on pg_catalog.unnest f  (cost=0.00..1.50 rows=100 width=96)   Output: (((lv)::text || op) || (rv)::text)  

Re: [HACKERS] [ADMIN] Unable to drop role

2010-08-24 Thread Robert Haas
On Tue, Aug 24, 2010 at 1:19 PM, Tom Lane t...@sss.pgh.pa.us wrote: Alvaro Herrera alvhe...@commandprompt.com writes: Happy to change it to whatever is deemed appropriate.  privileges for %s sounds good; I'll do that unless somebody comes up with a better idea which outvotes this one.

[HACKERS] Backups from the standby (Incrementally Updated Backups), open item

2010-08-24 Thread Josh Berkus
All, I've been looking at the open item which belongs with this doc: http://www.postgresql.org/docs/9.0/static/backup-incremental-updated.html ... and my viewpoint is that the procedure described should be *cut* from the official docs and put on the Wiki, with warnings. It's simply far too

Re: [HACKERS] Backups from the standby (Incrementally Updated Backups), open item

2010-08-24 Thread Bruce Momjian
Josh Berkus wrote: All, I've been looking at the open item which belongs with this doc: http://www.postgresql.org/docs/9.0/static/backup-incremental-updated.html ... and my viewpoint is that the procedure described should be *cut* from the official docs and put on the Wiki, with

Re: [HACKERS] EXPLAIN doesn't show the actual function expression for FunctionScan

2010-08-24 Thread David E. Wheeler
On Aug 24, 2010, at 10:21 AM, Tom Lane wrote: This may be the ultimate bike-shed but Wouldn't this be clearer the other way around? I generally think input comes first and then output. The order was bothering me a bit too, but there's a generic decision in there that the tlist is shown

Re: [HACKERS] EXPLAIN doesn't show the actual function expression for FunctionScan

2010-08-24 Thread Tom Lane
David E. Wheeler da...@kineticode.com writes: On Aug 24, 2010, at 10:21 AM, Tom Lane wrote: The order was bothering me a bit too, but there's a generic decision in there that the tlist is shown before any node-type-specific items. Not sure that we want to move it to the bottom for all of them.

Re: [HACKERS] About debug two versions of postgresql in eclipse

2010-08-24 Thread Nicolas Barbier
2010/8/24 Pei He hepeim...@gmail.com: I want to run two different versions of postgresql-8.2.5 under eclipse. But, it requires me to change PGDATA and LD_LIBRARY_PATH to switch. Moreover, to let eclipse know the changes, I need to modify .profile under my home folder, and log out and log

Re: [HACKERS] About debug two versions of postgresql in eclipse

2010-08-24 Thread Pei He
Thanks, Nicolas. It works. -- Pei On Tue, Aug 24, 2010 at 2:38 PM, Nicolas Barbier nicolas.barb...@gmail.com wrote: 2010/8/24 Pei He hepeim...@gmail.com: I want to run two different versions of postgresql-8.2.5 under eclipse. But, it requires me to change PGDATA and LD_LIBRARY_PATH to

Re: [HACKERS] Backups from the standby (Incrementally Updated Backups), open item

2010-08-24 Thread Bruce Momjian
Bruce Momjian wrote: Josh Berkus wrote: All, I've been looking at the open item which belongs with this doc: http://www.postgresql.org/docs/9.0/static/backup-incremental-updated.html ... and my viewpoint is that the procedure described should be *cut* from the official docs and

Re: [HACKERS] [BUGS] BUG #5305: Postgres service stops when closing Windows session

2010-08-24 Thread Magnus Hagander
On Tue, Aug 24, 2010 at 15:58, Tom Lane t...@sss.pgh.pa.us wrote: Bruce Momjian br...@momjian.us writes: Robert Haas wrote: Yeah, that seems very plausible, although exactly how to verify I don't know. And here is confirmation from the Microsoft web site:       In some instances, calling

Re: [HACKERS] [BUGS] BUG #5305: Postgres service stops when closing Windows session

2010-08-24 Thread Bruce Momjian
Magnus Hagander wrote: On Tue, Aug 24, 2010 at 15:58, Tom Lane t...@sss.pgh.pa.us wrote: Bruce Momjian br...@momjian.us writes: Robert Haas wrote: Yeah, that seems very plausible, although exactly how to verify I don't know. And here is confirmation from the Microsoft web site: ?

Re: [HACKERS] [BUGS] BUG #5305: Postgres service stops when closing Windows session

2010-08-24 Thread Magnus Hagander
On Tue, Aug 24, 2010 at 21:14, Bruce Momjian br...@momjian.us wrote: Magnus Hagander wrote: On Tue, Aug 24, 2010 at 15:58, Tom Lane t...@sss.pgh.pa.us wrote: Bruce Momjian br...@momjian.us writes: Robert Haas wrote: Yeah, that seems very plausible, although exactly how to verify I don't

Re: [HACKERS] Backups from the standby (Incrementally Updated Backups), open item

2010-08-24 Thread Marko Tiikkaja
On 2010-08-24 9:04 PM +0300, Josh Berkus wrote: ... and my viewpoint is that the procedure described should be *cut* from the official docs and put on the Wiki, with warnings. It's simply far too hackish and dependant on DBA understanding of PostgreSQL internals to belong in the main docs. In

Re: [HACKERS] [BUGS] BUG #5305: Postgres service stops when closing Windows session

2010-08-24 Thread Robert Haas
On Tue, Aug 24, 2010 at 3:10 PM, Magnus Hagander mag...@hagander.net wrote: On Tue, Aug 24, 2010 at 15:58, Tom Lane t...@sss.pgh.pa.us wrote: Bruce Momjian br...@momjian.us writes: Robert Haas wrote: Yeah, that seems very plausible, although exactly how to verify I don't know. And here is

Re: [HACKERS] [BUGS] BUG #5305: Postgres service stops when closing Windows session

2010-08-24 Thread Magnus Hagander
On Tue, Aug 24, 2010 at 21:39, Robert Haas robertmh...@gmail.com wrote: On Tue, Aug 24, 2010 at 3:10 PM, Magnus Hagander mag...@hagander.net wrote: On Tue, Aug 24, 2010 at 15:58, Tom Lane t...@sss.pgh.pa.us wrote: Bruce Momjian br...@momjian.us writes: Robert Haas wrote: Yeah, that seems very

Re: [HACKERS] Backups from the standby (Incrementally Updated Backups), open item

2010-08-24 Thread Tom Lane
Marko Tiikkaja marko.tiikk...@cs.helsinki.fi writes: Is this method not reliable then? Can something go wrong even if the user does exactly what the documentation says? It is not. This whole discussion started from somebody reporting that it didn't work. regards, tom

Re: [HACKERS] gSoC add MERGE command new patch -- merge_v104

2010-08-24 Thread Heikki Linnakangas
On 24/08/10 16:35, Boxuan Zhai wrote: Hi, I finished the MERGE on inheritance tables. Now comes the merge_v201 Oh, great! That means that all the known issues are fixed now, and all that's left is fixing any issues raised in review. I've added this to the September commitfest, but I hope

Re: [HACKERS] Backups from the standby (Incrementally Updated Backups), open item

2010-08-24 Thread Josh Berkus
On 8/24/10 12:42 PM, Tom Lane wrote: Marko Tiikkaja marko.tiikk...@cs.helsinki.fi writes: Is this method not reliable then? Can something go wrong even if the user does exactly what the documentation says? It is not. This whole discussion started from somebody reporting that it didn't

Re: [HACKERS] [BUGS] BUG #5305: Postgres service stops when closing Windows session

2010-08-24 Thread Robert Haas
On Tue, Aug 24, 2010 at 9:58 AM, Tom Lane t...@sss.pgh.pa.us wrote: Bruce Momjian br...@momjian.us writes: Robert Haas wrote: Yeah, that seems very plausible, although exactly how to verify I don't know. And here is confirmation from the Microsoft web site:       In some instances,

Re: [HACKERS] Backups from the standby (Incrementally Updated Backups), open item

2010-08-24 Thread Joshua D. Drake
On Tue, 2010-08-24 at 13:44 -0700, Josh Berkus wrote: On 8/24/10 12:42 PM, Tom Lane wrote: Marko Tiikkaja marko.tiikk...@cs.helsinki.fi writes: Is this method not reliable then? Can something go wrong even if the user does exactly what the documentation says? It is not. This whole

Re: [HACKERS] Backups from the standby (Incrementally Updated Backups), open item

2010-08-24 Thread Robert Haas
On Tue, Aug 24, 2010 at 4:44 PM, Josh Berkus j...@agliodbs.com wrote: On 8/24/10 12:42 PM, Tom Lane wrote: Marko Tiikkaja marko.tiikk...@cs.helsinki.fi writes: Is this method not reliable then?  Can something go wrong even if the user does exactly what the documentation says? It is not.  

Re: [HACKERS] gSoC add MERGE command new patch -- merge_v104

2010-08-24 Thread Andres Freund
On Tue, Aug 24, 2010 at 11:02:41PM +0300, Heikki Linnakangas wrote: On 24/08/10 16:35, Boxuan Zhai wrote: Hi, I finished the MERGE on inheritance tables. Now comes the merge_v201 Oh, great! That means that all the known issues are fixed now, and all that's left is fixing any issues raised

Re: [HACKERS] [BUGS] BUG #5305: Postgres service stops when closing Windows session

2010-08-24 Thread Tom Lane
Robert Haas robertmh...@gmail.com writes: There is some amount of user code (I'm not sure preceisely how much) that runs after shared memory is mapped and before the deadman switch is engaged. Er ... what would you define as user code? The deadman switch is engaged at the point where we

Re: [HACKERS] git: uh-oh

2010-08-24 Thread Robert Haas
On Mon, Aug 23, 2010 at 4:50 AM, Magnus Hagander mag...@hagander.net wrote: If we need to do it at all. Tom's latest lookover indicates that he thinks it may be good the way it is, and we need some more detailed checks. I know Robert has said he wants to dedicate some time to doing such checks

[HACKERS] HS/SR on AIX

2010-08-24 Thread Steve Singer
I think I've been able to reproduce the issue floating around with streaming replication on AIX. LOG: could not bind IPv6 socket: The socket name is already in use. HINT: Is another postmaster already running on port 5433? If not, wait a few seconds and retry. LOG: database system was

[HACKERS] why restrict role public but not Public?

2010-08-24 Thread Alvaro Herrera
Hi, I just noticed that we restrict creation of a role named public, but this is case-sensitive -- i.e. we don't restrict roles named PUBLIC, etc. Is this intended? -- Álvaro Herrera alvhe...@alvh.no-ip.org -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes

Re: [HACKERS] why restrict role public but not Public?

2010-08-24 Thread Tom Lane
Alvaro Herrera alvhe...@alvh.no-ip.org writes: I just noticed that we restrict creation of a role named public, but this is case-sensitive -- i.e. we don't restrict roles named PUBLIC, etc. Is this intended? Yes. If you had a role named that, you might think that GRANT whatever TO

Re: [HACKERS] why restrict role public but not Public?

2010-08-24 Thread Alvaro Herrera
Excerpts from Tom Lane's message of mar ago 24 19:04:14 -0400 2010: Alvaro Herrera alvhe...@alvh.no-ip.org writes: I just noticed that we restrict creation of a role named public, but this is case-sensitive -- i.e. we don't restrict roles named PUBLIC, etc. Is this intended? Yes. If

Re: [HACKERS] why restrict role public but not Public?

2010-08-24 Thread Tom Lane
Alvaro Herrera alvhe...@commandprompt.com writes: The reason I'm asking is that I'm trying to allow public to be passed to has_table_privileges(), and have it check for the PUBLIC pseudo-role. Originally I had coded it using pg_strcasecmp() on the grounds that any case should refer to this.

Re: [HACKERS] HS/SR on AIX

2010-08-24 Thread Tom Lane
Steve Singer ssin...@ca.afilias.info writes: I think I've been able to reproduce the issue floating around with streaming replication on AIX. Excellent, because we weren't getting much from the original reporter. This worked fine with beta2 but now seems to be an issue on beta4. If I do

Re: [HACKERS] gSoC add MERGE command new patch -- merge_v104

2010-08-24 Thread Boxuan Zhai
On Wed, Aug 25, 2010 at 4:56 AM, Andres Freund and...@anarazel.de wrote: On Tue, Aug 24, 2010 at 11:02:41PM +0300, Heikki Linnakangas wrote: On 24/08/10 16:35, Boxuan Zhai wrote: Hi, I finished the MERGE on inheritance tables. Now comes the merge_v201 Oh, great! That means that all

[HACKERS] No documentation for filtering dictionary feature?

2010-08-24 Thread Tom Lane
There's an entry in the 9.0 release notes saying that we've got filtering dictionaries now. Cool, but I don't see any documentation of the feature in textsearch.sgml. Shouldn't there be some? regards, tom lane -- Sent via pgsql-hackers mailing list

Re: [HACKERS] git: uh-oh

2010-08-24 Thread Robert Haas
On Fri, Aug 20, 2010 at 1:56 PM, Max Bowsher m...@f2s.com wrote: My guess at this point is that there may be a (very old?) version of cvs which, when adding a file to a branch, actually misrecorded the file as having existed on the branch from the moment it was first added to trunk - this

Re: [HACKERS] [BUGS] BUG #5305: Postgres service stops when closing Windows session

2010-08-24 Thread Robert Haas
On Tue, Aug 24, 2010 at 5:11 PM, Tom Lane t...@sss.pgh.pa.us wrote: Robert Haas robertmh...@gmail.com writes: There is some amount of user code (I'm not sure preceisely how much) that runs after shared memory is mapped and before the deadman switch is engaged. Er ... what would you define as

Re: [HACKERS] gSoC add MERGE command new patch -- merge_v104

2010-08-24 Thread Robert Haas
On Tue, Aug 24, 2010 at 4:56 PM, Andres Freund and...@anarazel.de wrote: Whats the plan to go forward at that subject? I think the patch needs to lock tables exclusively (the pg level, not access exclusive) as long as there is no additional handling... That sounds like it might cause more

Re: [HACKERS] SQLSTATE of notice PGresult

2010-08-24 Thread Robert Haas
On Fri, Aug 20, 2010 at 11:05 AM, Euler Taveira de Oliveira eu...@timbira.com wrote: Dmitriy Igrishin escreveu:   /* NOT presents - NULL. Why not 0 ? */   const char* sqlstate = Pg::PQresultErrorField(res, PG_DIAG_SQLSTATE); That's because the protocol doesn't set error field when the

Re: [HACKERS] security hook on authorization

2010-08-24 Thread KaiGai Kohei
I tried to revise the patch. It allows plugins to get control next to client authentication, but before returning the status to users. This change enables plugins which should be invoked on authentication failed to utilize this hook, not only assignment of session security label. At the same

Re: [HACKERS] [BUGS] BUG #5305: Postgres service stops when closing Windows session

2010-08-24 Thread David Fetter
On Tue, Aug 24, 2010 at 08:17:15PM -0400, Robert Haas wrote: On Tue, Aug 24, 2010 at 5:11 PM, Tom Lane t...@sss.pgh.pa.us wrote: Robert Haas robertmh...@gmail.com writes: There is some amount of user code (I'm not sure preceisely how much) that runs after shared memory is mapped and before

Re: [HACKERS] gSoC add MERGE command new patch -- merge_v104

2010-08-24 Thread David Fetter
On Wed, Aug 25, 2010 at 08:11:18AM +0800, Boxuan Zhai wrote: On Wed, Aug 25, 2010 at 4:56 AM, Andres Freund and...@anarazel.de wrote: On Tue, Aug 24, 2010 at 11:02:41PM +0300, Heikki Linnakangas wrote: On 24/08/10 16:35, Boxuan Zhai wrote: Hi, I finished the MERGE on inheritance

Re: [HACKERS] SQLSTATE of notice PGresult

2010-08-24 Thread Euler Taveira de Oliveira
Robert Haas escreveu: On Fri, Aug 20, 2010 at 11:05 AM, Euler Taveira de Oliveira eu...@timbira.com wrote: Dmitriy Igrishin escreveu: /* NOT presents - NULL. Why not 0 ? */ const char* sqlstate = Pg::PQresultErrorField(res, PG_DIAG_SQLSTATE); That's because the protocol doesn't set

Re: [HACKERS] SQLSTATE of notice PGresult

2010-08-24 Thread Robert Haas
On Tue, Aug 24, 2010 at 9:44 PM, Euler Taveira de Oliveira eu...@timbira.com wrote: Robert Haas escreveu: On Fri, Aug 20, 2010 at 11:05 AM, Euler Taveira de Oliveira eu...@timbira.com wrote: Dmitriy Igrishin escreveu:   /* NOT presents - NULL. Why not 0 ? */   const char* sqlstate =

Re: [HACKERS] SQLSTATE of notice PGresult

2010-08-24 Thread Tom Lane
Robert Haas robertmh...@gmail.com writes: I suppose we could change the function to return 0 always when the operation is not an error or warning report, rather than NULL, but certainly we wouldn't want to include those bytes in *every* success message, so they'd have to be something that

Re: [HACKERS] SQLSTATE of notice PGresult

2010-08-24 Thread Euler Taveira de Oliveira
Robert Haas escreveu: It appears to me that it already is documented. The very first sentence of the documentation reads: Returns an individual field of an error report. And a few sentences later it says: NULL is returned if the PGresult is not an error or warning result I'm

Re: [HACKERS] SQLSTATE of notice PGresult

2010-08-24 Thread Euler Taveira de Oliveira
Tom Lane escreveu: The real issue here is that there are no SQLSTATEs assigned for any error/warning conditions generated internally in libpq. Did you mean successful conditions? Only warning/error conditions produce a SQLSTATE. As far as this particular example goes, I think it's highly

Re: [HACKERS] git: uh-oh

2010-08-24 Thread Tom Lane
Robert Haas robertmh...@gmail.com writes: On Fri, Aug 20, 2010 at 1:56 PM, Max Bowsher m...@f2s.com wrote: My guess at this point is that there may be a (very old?) version of cvs which, when adding a file to a branch, actually misrecorded the file as having existed on the branch from the

Re: [HACKERS] SQLSTATE of notice PGresult

2010-08-24 Thread Tom Lane
Euler Taveira de Oliveira eu...@timbira.com writes: What I'm thinking is something like You didn't actually read what I said, did you? That patch will have precisely zero effect on the OP's example. regards, tom lane -- Sent via pgsql-hackers mailing list

[HACKERS] Performance Farm Release

2010-08-24 Thread Luxenberg, Scott I.
Hey all, This is just my email to notify you all that the project I've been working on with Stephen, the PostgreSQL Performance Farm, has been released. As of now, it only supports 9.0, due to the use of workers. More details can be found in the readme. The Git repository is located here:

Re: [HACKERS] git: uh-oh

2010-08-24 Thread Tom Lane
Robert Haas robertmh...@gmail.com writes: 1. The new conversion seems to have stolen the apostrophe from D'Arcy J.M. Cain da...@druid.net, rendering him DArcy J.M. Cain da...@druid.net. Yeah, I see that too. It's probably bad input rather than the converter's fault ;-) 2. Any non-ASCII

Re: [HACKERS] patch: Add JSON datatype to PostgreSQL (GSoC, WIP)

2010-08-24 Thread Itagaki Takahiro
Hi, I start to review JSON patch. On Fri, Aug 13, 2010 at 7:33 PM, Joseph Adams joeyadams3.14...@gmail.com wrote: Updated patch:  the JSON code has all been moved into core, so this patch is now for a built-in data type. I think the patch can be split into two pieces: 1. Basic I/O support

Re: [HACKERS] security label support, part.2

2010-08-24 Thread KaiGai Kohei
7. I think we need to write and include in the fine documentation some big picture documentation about enhanced security providers. Of course, we have to decide what we want to say. But the SECURITY LABEL documentation is just kind of hanging out there in space right now; it needs to connect