Re: [HACKERS] weird buildfarm failures on arm/mipsel and --with-tcl

2007-01-08 Thread Stefan Kaltenbrunner
Tom Lane wrote: > Stefan Kaltenbrunner <[EMAIL PROTECTED]> writes: >> one of my new buildfarm boxes (an Debian/Etch based ARM box) is >> sometimes failing to stop the database during the regression tests: > >> http://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=quagga&dt=2007-01-08%2003:03:03 >

Re: [HACKERS] [COMMITTERS] pgsql: Widen the money type to 64 bits.

2007-01-08 Thread Tom Lane
"Magnus Hagander" <[EMAIL PROTECTED]> writes: > i'm sure it got stuck in the maia filter (happens all the time with > delays of a couple of hours up to several days) or in the moderator > steve (if the sender was'nt on the list allow. can happen since it was > a long time since he last committed).

Re: [HACKERS] [COMMITTERS] pgsql: Widen the money type to 64 bits.

2007-01-08 Thread Magnus Hagander
> > > This commit message is rather old, I remarked on its absence earlier, but > > > better late than never I guess ;) > > > > Not sure what you mean. I just committed it recently. I held off > > until after the release as requested. What do you mean by "old?" > > You committed it five days a

Re: [HACKERS] is "add MOVE point" in ToDo valid still?

2007-01-08 Thread Pavel Stehule
Pavel Stehule wrote: > Hello, > > I am working on scrollable cursors, and MOVE statement has relation to it. You mean we should add MOVE to pl/pgsql. Yes, still on the TODO list, and I don't think it is completed --- not sure why it is so hard. -- It's simply task, but it needs some enhance

Re: [HACKERS] [PATCHES] vcbuild optional packages

2007-01-08 Thread Tom Lane
Magnus Hagander <[EMAIL PROTECTED]> writes: > This patch fixes vcbuild so you can build without OpenSSL and libz > should you want to. This disables the sslinfo and pgcrypto modules > because they require the libraries to build at all. Applied. regards, tom lane -

Re: [HACKERS] [PATCHES] README for vcbuild

2007-01-08 Thread Tom Lane
Magnus Hagander <[EMAIL PROTECTED]> writes: >>> Attached is a simple README file (part from Dave P, if my memory is not >>> mistaken) for the vcbuild stuff. Please put in src/tools/msvc. Applied, with Mark K's fix and a bit of polishing of my own. regards, tom lane --

Re: [HACKERS] ideas for auto-processing patches

2007-01-08 Thread Michael Glaesemann
On Jan 8, 2007, at 19:25 , Jim C. Nasby wrote: Actually, I see point in both... I'd think you'd want to know if a patch worked against the CVS checkout it was written against. Regardless, it's unlikely that the patch was tested against all of the platforms available on the build farm. If

Re: [HACKERS] [COMMITTERS] pgsql: Widen the money type to 64 bits.

2007-01-08 Thread Bruce Momjian
D'Arcy J.M. Cain wrote: > On Sun, 7 Jan 2007 21:04:09 -0800 (PST) > Jeremy Drake <[EMAIL PROTECTED]> wrote: > > This commit message is rather old, I remarked on its absence earlier, but > > better late than never I guess ;) > > Not sure what you mean. I just committed it recently. I held off > u

Re: [HACKERS] is "add MOVE point" in ToDo valid still?

2007-01-08 Thread Bruce Momjian
Pavel Stehule wrote: > Hello, > > I am working on scrollable cursors, and MOVE statement has relation to it. You mean we should add MOVE to pl/pgsql. Yes, still on the TODO list, and I don't think it is completed --- not sure why it is so hard. -- Bruce Momjian [EMAIL PROTECTED] Enterpri

Re: [HACKERS] [PATCHES] SGML index build fix

2007-01-08 Thread Bruce Momjian
Tom Lane wrote: > Bruce Momjian <[EMAIL PROTECTED]> writes: > > Peter Eisentraut wrote: > >> The problem is that this requires two runs even to proof the documentation, > >> which I think no one wants. > > > So what would the API be to signal you want a draft build? > > gmake DRAFT="Y" html >

Re: [HACKERS] pltcl regression failures with ORDER BY ... USING change

2007-01-08 Thread Tom Lane
Jeremy Drake <[EMAIL PROTECTED]> writes: > It looks like pltcl regression tests are failing due to the recent ORDER > BY ... USING change. Ooops ... I checked the contrib regression tests but it didn't occur to me to try pltcl :-( regards, tom lane ---

Re: [PATCHES] [HACKERS] [Fwd: Index Advisor]

2007-01-08 Thread Gurjeet Singh
On 1/8/07, Tom Lane <[EMAIL PROTECTED]> wrote: (This is not a statement that I approve of the specific plugin hooks proposed --- I don't particularly. But if we can come up with something a bit cleaner, that's how I'd approach it.) I have another idea for making the hooks a bit more cleaner;

Re: [HACKERS] weird buildfarm failures on arm/mipsel and --with-tcl

2007-01-08 Thread Tom Lane
Stefan Kaltenbrunner <[EMAIL PROTECTED]> writes: > one of my new buildfarm boxes (an Debian/Etch based ARM box) is > sometimes failing to stop the database during the regression tests: > http://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=quagga&dt=2007-01-08%2003:03:03 > this only seems to ha

[HACKERS] pltcl regression failures with ORDER BY ... USING change

2007-01-08 Thread Jeremy Drake
It looks like pltcl regression tests are failing due to the recent ORDER BY ... USING change. http://www.pgbuildfarm.org/cgi-bin/show_log.pl?nm=mongoose&dt=2007-01-09%2002:30:01 -- Horse sense is the thing a horse has which keeps it from betting on people. -- W. C. Fields --

Re: [HACKERS] Load distributed checkpoint

2007-01-08 Thread Takayuki Tsunakawa
Happy new year From: "Simon Riggs" <[EMAIL PROTECTED]> > Have you tried setting deadline scheduler on the WAL device and CFQ on > the data device? That should allow the I/Os to move through different > queues and prevent interference. No, I've not tried yet. Inaam-san told me that Linux had a fe

Re: [HACKERS] ideas for auto-processing patches

2007-01-08 Thread Jim C. Nasby
On Fri, Jan 05, 2007 at 11:02:32PM -0600, Andrew Dunstan wrote: > Tom Lane wrote: > > "Andrew Dunstan" <[EMAIL PROTECTED]> writes: > >> Jim Nasby wrote: > >>> More important, I see no reason to tie applying patches to pulling > >>> from CVS. In fact, I think it's a bad idea: you want to build just

Re: [HACKERS] [PATCHES] SGML index build fix

2007-01-08 Thread Tom Lane
Bruce Momjian <[EMAIL PROTECTED]> writes: > Peter Eisentraut wrote: >> The problem is that this requires two runs even to proof the documentation, >> which I think no one wants. > So what would the API be to signal you want a draft build? > gmake DRAFT="Y" html I'd vote for gmake d

[HACKERS] Strange sort operators vs the regression tests

2007-01-08 Thread Tom Lane
There was some discussion recently of simplifying the sorting code (and hopefully making it a tad faster) by eliminating support for using randomly-chosen operators in ORDER BY ... USING, and requiring USING to specify an operator that is the < or > member of some btree opclass. This strikes me as

Re: [HACKERS] -f option for pg_dumpall

2007-01-08 Thread Andreas Pflug
Jim C. Nasby wrote: > > > It might make sense to provide a programmatic interface to pg_dump to > provide tools like pgAdmin more flexibility. Are you talking about "pg_dump in a lib"? Certainly a good idea, because it allows better integration (e.g. progress bar). > But it certainly doesn't make

Re: [HACKERS] 8.3 pending patch queue

2007-01-08 Thread Bruce Momjian
Devrim GUNDUZ wrote: -- Start of PGP signed section. > Hi Bruce, > > On Mon, 2007-01-08 at 11:35 -0500, Bruce Momjian wrote: > > > OK, naming suggestions? > > BTW, why do you keep those pages in your homepage, but not in > postgresql.org? Just wondering. > > --and personally, I'd prefer to see

Re: [HACKERS] 8.3 pending patch queue

2007-01-08 Thread Devrim GUNDUZ
Hi Bruce, On Mon, 2007-01-08 at 11:35 -0500, Bruce Momjian wrote: > OK, naming suggestions? BTW, why do you keep those pages in your homepage, but not in postgresql.org? Just wondering. --and personally, I'd prefer to see them in our (PG) web page. Regards, -- The PostgreSQL Company - Command

Re: [HACKERS] -f option for pg_dumpall

2007-01-08 Thread Jim C. Nasby
On Fri, Jan 05, 2007 at 08:05:26PM +, Dave Page wrote: > Andreas Pflug wrote: > >Dave Page wrote: > >>In pgAdmin we use pg_dump's -f option to write backup files. The IO > >>streams are redirected to display status and errors etc. in the GUI. > >> > >>In order to enhance the interface to allow

Re: [HACKERS] (SETOF) RECORD AS complex_type

2007-01-08 Thread David Fetter
On Mon, Jan 08, 2007 at 05:25:17PM +0200, Hannu Krosing wrote: > Ühel kenal päeval, K, 2006-12-27 kell 14:06, kirjutas David Fetter: > > Folks, > > > > While using DBI-Link, I've noticed a little lacuna in how functions > > returning (SETOF) RECORD work, namely, that you have to cast them to > >

[HACKERS] weird buildfarm failures on arm/mipsel and --with-tcl

2007-01-08 Thread Stefan Kaltenbrunner
one of my new buildfarm boxes (an Debian/Etch based ARM box) is sometimes failing to stop the database during the regression tests: http://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=quagga&dt=2007-01-08%2003:03:03 this only seems to happen sometimes and only if --with-tcl is enabled on quagg

Re: [HACKERS] Mark/Restore and avoiding RandomAccess sorts

2007-01-08 Thread Jim C. Nasby
On Mon, Jan 08, 2007 at 10:37:25AM +, Heikki Linnakangas wrote: > >Simon Riggs wrote: > >>Implementing the variable mark/restore buffer as a dumb Tuplestore would > >>mean that the space usage of the Sort could in worst case go as high as > >>x2 total space. The worst case is where the inner sc

Re: [HACKERS] [COMMITTERS] pgsql: Widen the money type to 64 bits.

2007-01-08 Thread D'Arcy J.M. Cain
On Sun, 7 Jan 2007 21:04:09 -0800 (PST) Jeremy Drake <[EMAIL PROTECTED]> wrote: > This commit message is rather old, I remarked on its absence earlier, but > better late than never I guess ;) Not sure what you mean. I just committed it recently. I held off until after the release as requested.

Re: [HACKERS] 8.3 pending patch queue

2007-01-08 Thread Bruce Momjian
Heikki Linnakangas wrote: > Bruce Momjian wrote: > > Heikki Linnakangas wrote: > >> Bruce Momjian wrote: > >>> Simon Riggs wrote: > All have been awaiting review for at least a month (though in one case > the latest version is quite recent). They probably ought to be on the > hold qu

Re: [HACKERS] 8.3 pending patch queue

2007-01-08 Thread Bruce Momjian
Dave Page wrote: > Bruce Momjian wrote: > > > Right, because even the decision of whether they should be in the queue > > is a decision for us. The hold queue additions are less stringent than > > the main patch queue. > > Isn't that always the case though, not just after FF when the hold queue

Re: [PATCHES] [HACKERS] [Fwd: Index Advisor]

2007-01-08 Thread Simon Riggs
On Mon, 2007-01-08 at 12:16 -0500, Tom Lane wrote: > "Simon Riggs" <[EMAIL PROTECTED]> writes: > > On Mon, 2007-01-08 at 11:28 -0500, Bruce Momjian wrote: > >> The complex part of this is that the feature requires patches to the > >> backend, and has a /contrib component. > > > The plugin approach

Re: [HACKERS] 8.3 pending patch queue

2007-01-08 Thread Lukas Kahwe Smith
Andrew Dunstan wrote: The latter does not exist, AFAIK. Before feature freeze for cycle X, we don't usually hold patches for release X+1, as I understand it. In general, we should try to hold patches as little amount of time as possible. That way they don't go stale as easily. I did not follo

Re: [PATCHES] [HACKERS] [Fwd: Index Advisor]

2007-01-08 Thread Tom Lane
"Simon Riggs" <[EMAIL PROTECTED]> writes: > On Mon, 2007-01-08 at 11:28 -0500, Bruce Momjian wrote: >> The complex part of this is that the feature requires patches to the >> backend, and has a /contrib component. > The plugin approach is exactly what happened with the debugger. The > backend has

Re: [HACKERS] 8.3 pending patch queue

2007-01-08 Thread Andrew Dunstan
Heikki Linnakangas wrote: > I'm confused, So I see. > I thought the difference between the pgpatches queue and > the pgpatches_hold queue is the release the patch is targeted for. If > there's a third queue for patches that need review before being added to > another queue, could we have that vi

Re: [HACKERS] 8.3 pending patch queue

2007-01-08 Thread Heikki Linnakangas
Bruce Momjian wrote: Heikki Linnakangas wrote: Bruce Momjian wrote: Simon Riggs wrote: All have been awaiting review for at least a month (though in one case the latest version is quite recent). They probably ought to be on the hold queue; all are ready to be reviewed for final application/rej

Re: [PATCHES] [HACKERS] [Fwd: Index Advisor]

2007-01-08 Thread Simon Riggs
On Mon, 2007-01-08 at 11:28 -0500, Bruce Momjian wrote: > Simon Riggs wrote: > > On Sat, 2007-01-06 at 16:08 -0500, Bruce Momjian wrote: > > > > > I have looked over this patch, and it completes part of this TODO item: > > > > > > o Add SET PERFORMANCE_TIPS option to suggest INDEX, VACUUM

Re: [HACKERS] 8.3 pending patch queue

2007-01-08 Thread Dave Page
Bruce Momjian wrote: > Right, because even the decision of whether they should be in the queue > is a decision for us. The hold queue additions are less stringent than > the main patch queue. Isn't that always the case though, not just after FF when the hold queue starts getting activity again?

Re: [HACKERS] 8.3 pending patch queue

2007-01-08 Thread Bruce Momjian
Heikki Linnakangas wrote: > Bruce Momjian wrote: > > Simon Riggs wrote: > >> All have been awaiting review for at least a month (though in one case > >> the latest version is quite recent). They probably ought to be on the > >> hold queue; all are ready to be reviewed for final > >> application/rej

Re: [PATCHES] [HACKERS] [Fwd: Index Advisor]

2007-01-08 Thread Bruce Momjian
Simon Riggs wrote: > On Sat, 2007-01-06 at 16:08 -0500, Bruce Momjian wrote: > > > I have looked over this patch, and it completes part of this TODO item: > > > > o Add SET PERFORMANCE_TIPS option to suggest INDEX, VACUUM, VACUUM > > ANALYZE, and CLUSTER > > > It involves a pat

Re: [HACKERS] (SETOF) RECORD AS complex_type

2007-01-08 Thread Hannu Krosing
Ühel kenal päeval, K, 2006-12-27 kell 14:06, kirjutas David Fetter: > Folks, > > While using DBI-Link, I've noticed a little lacuna in how functions > returning (SETOF) RECORD work, namely, that you have to cast them to > explicit lists of columns, even when that list of columns corresponds > to a

Re: [HACKERS] [BUGS] BUG #2873: Function that returns an empty set with a 'not null' domain errors in 8.2 but not 8.1

2007-01-08 Thread Tom Lane
"Jonathan Hull" <[EMAIL PROTECTED]> writes: > The key feature for the error is that when a result structure (eg : pg_foo) > is defined with a domain type that is not null, only PG 8.2 errors if the > result is an empty set. The problem is explained well enough by this comment in plpgsql's code for

Re: [HACKERS] [PATCHES] Patch to log usage of temporary files

2007-01-08 Thread Bruce Momjian
Your patch has been added to the PostgreSQL unapplied patches list at: http://momjian.postgresql.org/cgi-bin/pgpatches It will be applied as soon as one of the PostgreSQL committers reviews and approves it. --- Bi

Re: [PATCHES] [HACKERS] [Fwd: Index Advisor]

2007-01-08 Thread Kenneth Marshall
One problem with only putting this information in the system logs is that when we provide database services to a member of our community we do not actually give them an account of the DB server or log server. This means that this very useful information would need to be passed through an intermedia

Re: [HACKERS] -f option for pg_dumpall

2007-01-08 Thread Dave Page
Dave Page wrote: >> I don't object to it in principle, but I think a bit more thought is >> needed as to what's the goal. A stupid "append" option would be enough >> for pg_dumpall's current capabilities (ie, text output only) --- but is >> it reasonable to consider generalizing -Fc and -Ft modes

Re: [HACKERS] Mark/Restore and avoiding RandomAccess sorts

2007-01-08 Thread Heikki Linnakangas
Bruce Momjian wrote: I saw no replies to this. Sounds like a good idea to me. (further comments below) Simon Riggs wrote: Implementing the variable mark/restore buffer as a dumb Tuplestore would mean that the space usage of the Sort could in worst case go as high as x2 total space. The wors

Re: [HACKERS] 8.3 pending patch queue

2007-01-08 Thread Heikki Linnakangas
Bruce Momjian wrote: Simon Riggs wrote: All have been awaiting review for at least a month (though in one case the latest version is quite recent). They probably ought to be on the hold queue; all are ready to be reviewed for final application/rejection. I'd hasten to add that none of those are

Re: [HACKERS] [PATCHES] SGML index build fix

2007-01-08 Thread Peter Eisentraut
Am Montag, 8. Januar 2007 05:10 schrieb Bruce Momjian: > Here is a patch that runs the build twice when HTML.index does not > exist, and once every time after that. This is not ideal, but it is a > start. The problem is that this requires two runs even to proof the documentation, which I think no

Re: [HACKERS] security definer default for some PL languages (SQL/PSM)?

2007-01-08 Thread Albe Laurenz
Peter Eisentraut wrote: >> SQL/PSM default for SQL procedures are SECURITY DEFINER (like views), > > I can't find this in the standard. Where did you get this > information? I only have a draft version of SQL:2003, which says in the 'Foundation' book, chapter 11.50 (""), about the "" (which can

[HACKERS] is "add MOVE point" in ToDo valid still?

2007-01-08 Thread Pavel Stehule
Hello, I am working on scrollable cursors, and MOVE statement has relation to it. Regards Pavel Stehule _ Chcete sdilet sve obrazky a hudbu s prateli? http://messenger.msn.cz/ ---(end of broadcast)