Re: [HACKERS] btree_gist (was: CommitFest progress - or lack thereof)

2011-02-12 Thread Oleg Bartunov
Stephen, what do you need for documentation ? From users point of view we add just knn support and all examples are available in btree_gist.sql and sql subdirectory. Contact me directly, if you have questions. Oleg On Fri, 11 Feb 2011, Stephen Frost wrote: * Robert Haas

Re: [HACKERS] pl/python tracebacks

2011-02-12 Thread Jan Urbański
On 12/02/11 04:12, Alex Hunsaker wrote: On Wed, Feb 9, 2011 at 02:10, Jan Urbański wulc...@wulczer.org wrote: On 06/02/11 20:12, Jan Urbański wrote: On 27/01/11 22:58, Jan Urbański wrote: On 23/12/10 14:56, Jan Urbański wrote: Here's a patch implementing traceback support for PL/Python

Re: [HACKERS] pl/python tracebacks

2011-02-12 Thread Alex Hunsaker
On Sat, Feb 12, 2011 at 01:50, Jan Urbański wulc...@wulczer.org wrote: On 12/02/11 04:12, Alex Hunsaker wrote: In PLy_traceback fname and prname look like they will leak (well as much as a palloc() in an error path can leak I suppose). But they're no palloc'd, no? fname is either a static

Re: [HACKERS] pl/python tracebacks

2011-02-12 Thread Jan Urbański
On 12/02/11 10:00, Alex Hunsaker wrote: On Sat, Feb 12, 2011 at 01:50, Jan Urbański wulc...@wulczer.org wrote: On 12/02/11 04:12, Alex Hunsaker wrote: In PLy_traceback fname and prname look like they will leak (well as much as a palloc() in an error path can leak I suppose). But they're no

[HACKERS] Re: [COMMITTERS] pgsql: Force strings passed to and from plperl to be in UTF8 encoding.

2011-02-12 Thread Alex Hunsaker
On Sun, Feb 6, 2011 at 15:31, Andrew Dunstan and...@dunslane.net wrote: Force strings passed to and from plperl to be in UTF8 encoding. String are converted to UTF8 on the way into perl and to the database encoding on the way back. This avoids a number of observed anomalies, and ensures Perl

Re: [HACKERS] pl/python explicit subtransactions

2011-02-12 Thread Jan Urbański
On 11/02/11 17:22, Steve Singer wrote: On 11-02-10 05:20 AM, Jan Urbański wrote: D'oh, I was thinking about whether it's safe to skip the internal subxact if you're in an implicit one and somehow I always convinced myself that since you eventually close the explicit one, it is. Obviously my

Re: [HACKERS] pl/python custom exceptions for SPI

2011-02-12 Thread Jan Urbański
On 11/02/11 10:53, Jan Urbański wrote: On 10/02/11 22:26, Steve Singer wrote: Here's an updated patch with documentation. It's an incremental patch on top of the latest explicit-subxacts version. Cheers, Jan diff --git a/doc/src/sgml/plpython.sgml b/doc/src/sgml/plpython.sgml index

Re: [HACKERS] SSI bug?

2011-02-12 Thread YAMAMOTO Takashi
hi, YAMAMOTO Takashi y...@mwd.biglobe.ne.jp wrote: it seems that PredicateLockTupleRowVersionLink sometimes create a loop of targets (it founds an existing 'newtarget' whose nextVersionOfRow chain points to the 'oldtarget') and it later causes CheckTargetForConflictsIn loop forever.

Re: [HACKERS] [Mingw-users] mingw64

2011-02-12 Thread Ralf Wildenhues
Hello, and sorry for the delay, * Peter Rosin wrote on Sat, Jan 29, 2011 at 02:26:24PM CET: Or is plain 'ar' used somewhere instead of 'x86_64-w64-mingw32-ar'? Automake outputs 'AR = ar' in Makefile.in for rules creating old libraries iff neither AC_PROG_LIBTOOL nor another method to define AR

Re: [HACKERS] Fwd: [JDBC] Weird issues when reading UDT from stored function

2011-02-12 Thread Lukas Eder
I had tried that before. That doesn't seem to change anything. JDBC still expects 6 OUT parameters, instead of just 1... 2011/2/11 Robert Haas robertmh...@gmail.com On Tue, Jan 25, 2011 at 2:39 AM, Lukas Eder lukas.e...@gmail.com wrote: So what you're suggesting is that the plpgsql code is

[HACKERS] pika failing since the per-column collation patch

2011-02-12 Thread Rémi Zara
Hi, Since the per-column collation patch went in, pika (NetBSD 5.1/mips) started failing consistently with this diff: *** /home/pgbuildfarm/workdir/HEAD/pgsql.15101/src/test/regress/expected/polymorphism.out Sat Feb 12 02:16:07 2011 ---

Re: [HACKERS] btree_gist (was: CommitFest progress - or lack thereof)

2011-02-12 Thread Stephen Frost
Oleg, * Oleg Bartunov (o...@sai.msu.su) wrote: what do you need for documentation ? From users point of view we add just knn support and all examples are available in btree_gist.sql and sql subdirectory. Contact me directly, if you have questions. It sure seems like

Re: [HACKERS] btree_gist (was: CommitFest progress - or lack thereof)

2011-02-12 Thread Robert Haas
On Sat, Feb 12, 2011 at 7:47 AM, Stephen Frost sfr...@snowman.net wrote: Oleg, * Oleg Bartunov (o...@sai.msu.su) wrote: what do you need for documentation ? From users point of view we add just knn support and all examples are available in btree_gist.sql and sql subdirectory. Contact me

Re: [HACKERS] multiset patch review

2011-02-12 Thread Stephen Frost
Itagaki, * Itagaki Takahiro (itagaki.takah...@gmail.com) wrote: On Sat, Feb 12, 2011 at 05:01, Stephen Frost sfr...@snowman.net wrote: What does the spec say about this, if anything?  Is that required by spec, or is the spec not relevant to this because MULTISETs are only one

Re: [HACKERS] Change pg_last_xlog_receive_location not to move backwards

2011-02-12 Thread Robert Haas
On Fri, Feb 11, 2011 at 12:52 PM, Stephen Frost sfr...@snowman.net wrote: * Robert Haas (robertmh...@gmail.com) wrote: Actually... wait a minute.  Now that I'm thinking about this a little more, I'm not so convinced.  If we leave this the way is, and just paper over the problem using the

Re: [HACKERS] is_absolute_path incorrect on Windows

2011-02-12 Thread Bruce Momjian
Bruce Momjian wrote: Bruce Momjian wrote: Tom Lane wrote: Bruce Momjian br...@momjian.us writes: Tom Lane wrote: Bruce Momjian br...@momjian.us writes: I have reviewed is_absolute_path() and have implemented path_is_relative_and_below_cwd() to cleanly handle cases like

Re: [HACKERS] ALTER TYPE 2: skip already-provable no-work rewrites

2011-02-12 Thread Noah Misch
On Fri, Feb 11, 2011 at 02:49:27PM -0500, Robert Haas wrote: You might want to consider a second boolean in lieu of a three way enum. I'm not sure if that's cleaner but if it lets you write: if (blah) at-verify = true; instead of: if (blah) at-worklevel = Min(at-worklevel,

Re: [HACKERS] Change pg_last_xlog_receive_location not to move backwards

2011-02-12 Thread Stephen Frost
* Robert Haas (robertmh...@gmail.com) wrote: I think a big part of the problem here is that we have wildly inconsistent terminology for no especially good reason. Agreed. Thoughts, comments? I thought about this for a bit and agree w/ your suggestions. So, +1 from me. Thanks,

Re: [HACKERS] btree_gist (was: CommitFest progress - or lack thereof)

2011-02-12 Thread Oleg Bartunov
Stephen, On Sat, 12 Feb 2011, Stephen Frost wrote: Oleg, * Oleg Bartunov (o...@sai.msu.su) wrote: what do you need for documentation ? From users point of view we add just knn support and all examples are available in btree_gist.sql and sql subdirectory. Contact me directly, if you have

[HACKERS] Extensions vs PGXS' MODULE_PATHNAME handling

2011-02-12 Thread Tom Lane
I've run into a small infelicity that was introduced by our recent round of redesign of the extensions feature. Specifically, if we have an installation script that is named like hstore-1.0.sql.in, then what pgxs.mk will substitute for MODULE_PATHNAME in it is $libdir/hstore-1.0 ... not exactly

Re: [HACKERS] SSI bug?

2011-02-12 Thread Kevin Grittner
YAMAMOTO Takashi y...@mwd.biglobe.ne.jp wrote: i have seen this actually happen. i've confirmed the creation of the loop with the attached patch. it's easily reproducable with my application. i can provide the full source code of my application if you want. (but it isn't easy to run unless

Re: [HACKERS] Change pg_last_xlog_receive_location not to move backwards

2011-02-12 Thread Simon Riggs
On Sat, 2011-02-12 at 09:32 -0500, Robert Haas wrote: On Fri, Feb 11, 2011 at 12:52 PM, Stephen Frost sfr...@snowman.net wrote: * Robert Haas (robertmh...@gmail.com) wrote: Actually... wait a minute. Now that I'm thinking about this a little more, I'm not so convinced. If we leave this

Re: [HACKERS] Change pg_last_xlog_receive_location not to move backwards

2011-02-12 Thread Simon Riggs
On Mon, 2011-01-31 at 16:12 +0900, Fujii Masao wrote: On Sun, Jan 30, 2011 at 10:44 AM, Jeff Janes jeff.ja...@gmail.com wrote: I do not understand what doing so gets us. Say we previously received 2/3 of a WAL file, and replayed most of it. So now the shared buffers have data that has

Re: [HACKERS] Extensions vs PGXS' MODULE_PATHNAME handling

2011-02-12 Thread Dimitri Fontaine
Tom Lane t...@sss.pgh.pa.us writes: pgxs.mk will substitute for MODULE_PATHNAME in it is $libdir/hstore-1.0 ... not exactly what's wanted. This is because the transformation rule depends on $*, ie the base name of the input file. [...] On balance #3 seems the least bad, but I wonder if anyone

Re: [HACKERS] pika failing since the per-column collation patch

2011-02-12 Thread Peter Eisentraut
On lör, 2011-02-12 at 13:34 +0100, Rémi Zara wrote: Since the per-column collation patch went in, pika (NetBSD 5.1/mips) started failing consistently with this diff: *** /home/pgbuildfarm/workdir/HEAD/pgsql.15101/src/test/regress/expected/polymorphism.out Sat Feb 12 02:16:07 2011

Re: [HACKERS] SSI bug?

2011-02-12 Thread Kevin Grittner
I wrote: it seems likely that such a cycle might be related to this new code not properly allowing for some aspect of tuple cleanup. I found a couple places where cleanup could let these fall through the cracks long enough to get stale and still be around when a tuple ID is re-used, causing

Re: [HACKERS] btree_gist (was: CommitFest progress - or lack thereof)

2011-02-12 Thread Jeff Davis
On Sat, 2011-02-12 at 18:53 +0300, Oleg Bartunov wrote: It sure seems like http://www.postgresql.org/docs/9.0/static/btree-gist.html could be and should be improved, in general.. If this module is really still just a test bed for GiST, then perhaps it's not a big deal.. No, it's quite

Re: [HACKERS] Sorting. When?

2011-02-12 Thread mac_man2...@yahoo.it
So, invoking or not invoking sorting depends on different parameters of the specific DBMS, does it? This also means that it depends on the specific implementation of the Planner and, as a consequence, *on the specific DBMS*? I mean, different DBMS can chose differently on invoking sorting even

Re: [HACKERS] Sorting. When?

2011-02-12 Thread Pavel Stehule
2011/2/12 mac_man2...@yahoo.it mac_man2...@yahoo.it: So, invoking or not invoking sorting depends on different parameters of the specific DBMS, does it? This also means that it depends on the specific implementation of the Planner and, as a consequence, on the specific DBMS? I mean,

Re: [HACKERS] Sorting. When?

2011-02-12 Thread Kevin Grittner
mac_man2...@yahoo.it mac_man2...@yahoo.it wrote: So, invoking or not invoking sorting depends on different parameters of the specific DBMS, does it? In the right circumstances (where autovacuum or other maintenance jobs happen to run in the background at the right moment), two back-to-back

Re: [HACKERS] SQL/MED - file_fdw

2011-02-12 Thread Kevin Grittner
Noah Misch n...@leadboat.com wrote: I'd say, run them with this patch alone. The important thing is to not penalize existing COPY users. Incidentally, the did you want ... ? was a genuine question. I see very little performance risk here, so the tests could be quite cursory, even absent

Re: [HACKERS] Debian readline/libedit breakage

2011-02-12 Thread Dimitri Fontaine
Tom Lane t...@sss.pgh.pa.us writes: Less narrow-minded interpretation of GPL requirements, perhaps. (And yes, we have real lawyers on staff considering these issues.) If we really believe that the debian interpretation of the licence issue here is moot, surely the easiest action is to offer a

Re: [HACKERS] btree_gist (was: CommitFest progress - or lack thereof)

2011-02-12 Thread Oleg Bartunov
Oops, thank for remind ! Oleg On Sat, 12 Feb 2011, Tom Lane wrote: Jeff Davis pg...@j-davis.com writes: btree_gist is essentially required for exclusion constraints to be useful in a practical way. In fact, can you submit it for the next commitfest to be included in core? That would allow

Re: [HACKERS] btree_gist (was: CommitFest progress - or lack thereof)

2011-02-12 Thread Tom Lane
Jeff Davis pg...@j-davis.com writes: btree_gist is essentially required for exclusion constraints to be useful in a practical way. In fact, can you submit it for the next commitfest to be included in core? That would allow range types and exclusion constraints to be used out-of-the-box in

Re: [HACKERS] Extensions vs PGXS' MODULE_PATHNAME handling

2011-02-12 Thread Tom Lane
Dimitri Fontaine dimi...@2ndquadrant.fr writes: Tom Lane t...@sss.pgh.pa.us writes: pgxs.mk will substitute for MODULE_PATHNAME in it is $libdir/hstore-1.0 ... not exactly what's wanted. This is because the transformation rule depends on $*, ie the base name of the input file. A though that

Re: [HACKERS] Debian readline/libedit breakage

2011-02-12 Thread Magnus Hagander
On Sat, Feb 12, 2011 at 22:46, Dimitri Fontaine dimi...@2ndquadrant.fr wrote: Tom Lane t...@sss.pgh.pa.us writes: Less narrow-minded interpretation of GPL requirements, perhaps. (And yes, we have real lawyers on staff considering these issues.) If we really believe that the debian

[HACKERS] XMin Hot Standby Feedback patch

2011-02-12 Thread Daniel Farina
This is another bit of the syncrep patch split out. I will revisit the replication timeout one Real Soon, I promise -- but I have a couple things to do today that may delay that until the evening. https://github.com/fdr/postgres/commit/ad3ce9ac62f0e128d7d1fd20d47184f867056af1 Context diff

Re: [HACKERS] Extensions vs PGXS' MODULE_PATHNAME handling

2011-02-12 Thread Tom Lane
Dimitri Fontaine dimi...@2ndquadrant.fr writes: Tom Lane t...@sss.pgh.pa.us writes: Right, the basic difficulty here is exactly that in a Makefile that's building multiple shlibs, there is no easy way to decide which shlibs go with which sql scripts. The existing implementation essentially

Re: [HACKERS] SQL/MED - file_fdw

2011-02-12 Thread Noah Misch
On Sat, Feb 12, 2011 at 03:42:17PM -0600, Kevin Grittner wrote: In two hours of testing with a 90GB production database, the copy patch on top of HEAD ran 0.6% faster than HEAD for pg_dumpall (generating identical output files), but feeding that in to and empty cluster with psql ran 8.4%

Re: [HACKERS] Extensions vs PGXS' MODULE_PATHNAME handling

2011-02-12 Thread David E. Wheeler
On Feb 12, 2011, at 2:29 PM, Tom Lane wrote: I did think of another idea besides forbidding dash in extension names: what if we use double dash as the name/version separator, ie the naming conventions are like extension--version.control extension--version.sql

Re: [HACKERS] Extensions vs PGXS' MODULE_PATHNAME handling

2011-02-12 Thread Tom Lane
David E. Wheeler da...@kineticode.com writes: On Feb 12, 2011, at 2:29 PM, Tom Lane wrote: I did think of another idea besides forbidding dash in extension names: what if we use double dash as the name/version separator, +1 You might even consider mandating a double-dash between versions, so

Re: [HACKERS] Extensions vs PGXS' MODULE_PATHNAME handling

2011-02-12 Thread David E. Wheeler
On Feb 12, 2011, at 3:12 PM, Tom Lane wrote: David E. Wheeler da...@kineticode.com writes: On Feb 12, 2011, at 2:29 PM, Tom Lane wrote: I did think of another idea besides forbidding dash in extension names: what if we use double dash as the name/version separator, +1 You might even

Re: [HACKERS] Extensions vs PGXS' MODULE_PATHNAME handling

2011-02-12 Thread Tom Lane
David E. Wheeler da...@kineticode.com writes: On Feb 12, 2011, at 3:12 PM, Tom Lane wrote: Hm. I think we'd still have to disallow dash as the first or last character in a version name to make that unambiguous. Not sure it's worth the trouble. How likely is *that*? Not very, but the rules

Re: [HACKERS] Extensions vs PGXS' MODULE_PATHNAME handling

2011-02-12 Thread David E. Wheeler
On Feb 12, 2011, at 3:37 PM, Tom Lane wrote: How likely is *that*? Not very, but the rules are getting a bit complicated ... Doesn't seem complicated to me: 1. Use -- to separate extension name, old version, new version 2. Don't use - at the beginning or end of name or version number 3.

Re: [HACKERS] Debian readline/libedit breakage

2011-02-12 Thread Charles.McDevitt
charles.mcdev...@emc.com wrote: The GNU people will never be 100% satisfied by anything you do to psql, other than making it GPL. Readline is specifically licensed in a way to try to force this (but many disagree with their ability to force this). The GNU people are perfectly

Re: [HACKERS] Extensions vs PGXS' MODULE_PATHNAME handling

2011-02-12 Thread Dimitri Fontaine
Tom Lane t...@sss.pgh.pa.us writes: Right, the basic difficulty here is exactly that in a Makefile that's building multiple shlibs, there is no easy way to decide which shlibs go with which sql scripts. The existing implementation essentially relies on the base name of the sql script matching

Re: [HACKERS] Debian readline/libedit breakage

2011-02-12 Thread Dimitri Fontaine
Magnus Hagander mag...@hagander.net writes: On Sat, Feb 12, 2011 at 22:46, Dimitri Fontaine dimi...@2ndquadrant.fr wrote: If we really believe that the debian interpretation of the licence issue here is moot, surely the easiest action is to offer a debian package repository hosted in the

Re: [HACKERS] Debian readline/libedit breakage

2011-02-12 Thread Greg Smith
Dimitri Fontaine wrote: Now, what I think I would do about the core package is a quite simple backport of them, using Martin's excellent work. Do we want our own QA on them? If yes, I think I would need some help here, maybe with some build farm support for running from our debian packages

Re: [HACKERS] ALTER TYPE 2: skip already-provable no-work rewrites

2011-02-12 Thread Robert Haas
On Sat, Feb 12, 2011 at 10:45 AM, Noah Misch n...@leadboat.com wrote: That said, I've tried both constructions, and I marginally prefer the end result with AlteredTableInfo.verify.  I've inlined ATColumnChangeRequiresRewrite into ATPrepAlterColumnType; it would need to either pass back two