Re: [HACKERS] Concurrent MERGE

2010-08-05 Thread Kevin Grittner
Josh Berkus wrote: > Overall, you're missing the point: there are workarounds for all > of these things now. However, they are *workarounds*, which means > that they are awkward, expensive, and/or hard to administrate; > having predicate locks would make things much easier. Well, if some form

Re: [HACKERS] Drop one-argument string_agg? (was Re: [BUGS] string_agg delimiter having no effect with order by)

2010-08-05 Thread Tom Lane
Peter Eisentraut writes: > On tor, 2010-08-05 at 14:38 -0400, Tom Lane wrote: >> Huh? The functionality proposed for removal is only that of omitting >> an explicit delimiter argument for string_agg(). Since the default >> value (an empty string) doesn't seem to be the right thing all that >> of

Re: [HACKERS] PL/pgSQL EXECUTE '..' USING with unknown

2010-08-05 Thread Tom Lane
Heikki Linnakangas writes: > There's a little problem with EXECUTE USING when the parameters are of > type unknown (going back to 8.4 where EXECUTE USING was introduced): > do $$ > BEGIN >EXECUTE 'SELECT to_date($1, $2)' USING '17-DEC-80', 'DD-MON-YY'; > END; > $$; > ERROR: failed to find c

Re: [HACKERS] PL/pgSQL EXECUTE '..' USING with unknown

2010-08-05 Thread Andrew Dunstan
On 08/05/2010 05:11 PM, Tom Lane wrote: Heikki Linnakangas writes: There's a little problem with EXECUTE USING when the parameters are of type unknown (going back to 8.4 where EXECUTE USING was introduced): do $$ BEGIN EXECUTE 'SELECT to_date($1, $2)' USING '17-DEC-80', 'DD-MON-YY'; END;

Re: [HACKERS] Patch to show individual statement latencies in pgbench output

2010-08-05 Thread Florian Pflug
On Aug4, 2010, at 13:58 , Florian Pflug wrote: > On Aug3, 2010, at 21:16 , Greg Smith wrote: >>> That was a leftover of the trimming and comment skipping logic, which my >>> patch moves to process_command. >> >> I think there's still a trimming error here--line 195 of the new patch is >> now re

Re: [HACKERS] PL/pgSQL EXECUTE '..' USING with unknown

2010-08-05 Thread Tom Lane
Andrew Dunstan writes: > On 08/05/2010 05:11 PM, Tom Lane wrote: >> This example doesn't seem terribly compelling. Why would you bother >> using USING with constants? > In a more complex example you might use $1 in more than one place in the > query. Well, that's better than no justification,

Re: [HACKERS] Concurrent MERGE

2010-08-05 Thread Josh Berkus
On 8/5/10 1:59 PM, Kevin Grittner wrote: > Oh, and if deadlocks are that broken, it's a bit scary that we have > let that go. Is it the problem that technically intractable? Yes; it's a major project. Our detector works pretty well for deadlocks which are 2-process locks or even several processe

Re: [HACKERS] Concurrent MERGE

2010-08-05 Thread Tom Lane
Josh Berkus writes: > Yes; it's a major project. Our detector works pretty well for deadlocks > which are 2-process locks or even several processes all locking against > the same first process. However, triangular and quadralateral deadlocks > (which I've seen more than once) it completely cannot

Re: [HACKERS] Concurrent MERGE

2010-08-05 Thread Josh Berkus
> Hm? Please explain what you're talking about. Transaction A locks 1 and wants a lock on 2 Transaction B locks 2 and wants a lock on 3 Transaction C locks 3 and wants a lock on 1 I've never had the deadlock detector successfully deal with the above. Let alone a 4-way. > Not sure I believe thi

Re: [HACKERS] Initial review of xslt with no limits patch

2010-08-05 Thread Mike Fowler
Hi Pavel, On 02/08/10 09:21, Pavel Stehule wrote: Hello 2010/8/2 Mike Fowler: Hi Pavel, Currently your patch isn't applying to head, from the looks of things a function signature has changed. Can you update your patch please? yes - see attachment Thanks, the new patch applies cleanly. H

Re: [HACKERS] including backend ID in relpath of temp rels - updated patch

2010-08-05 Thread Tom Lane
Robert Haas writes: > [ BackendRelFileNode patch ] One thing that I find rather distressing about this is the 25% bloat in sizeof(SharedInvalidationMessage). Couldn't we avoid that? Is it really necessary to *ever* send an SI message for a backend-local rel? I agree that one needs to send relc

Re: [HACKERS] Concurrent MERGE

2010-08-05 Thread Mark Kirkwood
On 06/08/10 10:49, Josh Berkus wrote: Hm? Please explain what you're talking about. Transaction A locks 1 and wants a lock on 2 Transaction B locks 2 and wants a lock on 3 Transaction C locks 3 and wants a lock on 1 I've never had the deadlock detector successfully deal with the abov

Re: [HACKERS] Concurrent MERGE

2010-08-05 Thread Andres Freund
On Thu, Aug 05, 2010 at 03:49:05PM -0700, Josh Berkus wrote: > > > Hm? Please explain what you're talking about. > > Transaction A locks 1 and wants a lock on 2 > Transaction B locks 2 and wants a lock on 3 > Transaction C locks 3 and wants a lock on 1 > > I've never had the deadlock detector succ

Re: [HACKERS] Concurrent MERGE

2010-08-05 Thread Tom Lane
Josh Berkus writes: >> Hm? Please explain what you're talking about. > Transaction A locks 1 and wants a lock on 2 > Transaction B locks 2 and wants a lock on 3 > Transaction C locks 3 and wants a lock on 1 > I've never had the deadlock detector successfully deal with the above. > Let alone a 4

Re: [HACKERS] Concurrent MERGE

2010-08-05 Thread Josh Berkus
>> I've never had the deadlock detector successfully deal with the above. >> Let alone a 4-way. > Hm. I have seen 5way deadlocks getting resolved just recently. I can > find the relevant if you find it interesting, but I doubt it is. Ah, I didn't know that it was even *supposed* to resolve larger

Re: [HACKERS] MERGE Specification

2010-08-05 Thread Boxuan Zhai
Dear All, I have seen a lively discussion about the DO NOTING action in MERGE command. And, I think most people want it. So it will be added to my next patch. Before the implementation, I still have some questions to confirm: 1. If we have a DO NOTHING action specified, it should be the last WHE

[HACKERS] LockDatabaseObject vs. LockSharedObject

2010-08-05 Thread Robert Haas
It seems suspicious to me that LockSharedObject() calls AcceptInvalidationMessges() and LockDatabaseObject() does not. Since the only caller of LockSharedObject() at present is AcquireDeletionLock(), I'm not sure there's an observable bug here at the moment, but then again, I'm also not sure there

Re: [PATCH] Re: [HACKERS] Adding xpath_exists function

2010-08-05 Thread Alvaro Herrera
Excerpts from Mike Fowler's message of mar jun 29 06:37:28 -0400 2010: > After seeing some other posts in the last couple of days, I realised I > hadn't documented the function in the SGML. I have now done so, and > added a couple of tests with XML literals. Please find the patch > attached. No

Re: [HACKERS] Initial review of xslt with no limits patch

2010-08-05 Thread Pavel Stehule
2010/8/6 Mike Fowler : > Hi Pavel, > > On 02/08/10 09:21, Pavel Stehule wrote: >> >> Hello >> >> 2010/8/2 Mike Fowler: >>> >>> Hi Pavel, >>> >>> Currently your patch isn't applying to head, from the looks of things a >>> function signature has changed. Can you update your patch please? >>> >> >> ye

Re: [HACKERS] Initial review of xslt with no limits patch

2010-08-05 Thread Andrew Dunstan
On 08/05/2010 06:56 PM, Mike Fowler wrote: SELECT xslt_process('cim30400'::text, $$http://www.w3.org/1999/XSL/Transform"; version="1.0"> [snip] $$::text, 'n1=v1,n2=v2,n3=v3,n4=v4,n5=v5'::text) I haven't been paying attention to this, so sorry if this has been discussed before, but

[HACKERS] pgsql-hack...@news.hub.org 81% OFF on Pfizer!

2010-08-05 Thread pgsql-hackers
http://groups.yahoo.com/group/pedranmarcksld/message vielmehr an das Vorhandensein eines {688 Voraussetzung fur die Befreiung verlorener Gebiete} wenn auch noch so kleinen Restes dieses Volkes und Staates, der, im B

Re: [HACKERS] Initial review of xslt with no limits patch

2010-08-05 Thread Pavel Stehule
2010/8/6 Andrew Dunstan : > > > On 08/05/2010 06:56 PM, Mike Fowler wrote: >> >> SELECT >> xslt_process('cim30400'::text, >> $$http://www.w3.org/1999/XSL/Transform"; >> version="1.0"> >> >> > [snip] >> >> $$::text, 'n1=v1,n2=v2,n3=v3,n4=v4,n5=v5'::text) >> >> > > I haven't been paying attention to

Re: Review: Re: [PATCH] Re: [HACKERS] Adding xpath_exists function

2010-08-05 Thread Peter Eisentraut
On tis, 2010-07-27 at 16:33 -0700, David Fetter wrote: > * Do we already have it? > > Not really. There are kludges to accomplish these things, but > they're available mostly in the sense that a general-purpose > language allows you to write code to do anything a Turing machine >

Re: [HACKERS] Initial review of xslt with no limits patch

2010-08-05 Thread David Fetter
On Fri, Aug 06, 2010 at 05:57:37AM +0200, Pavel Stehule wrote: > 2010/8/6 Andrew Dunstan : > > On 08/05/2010 06:56 PM, Mike Fowler wrote: > >> SELECT > >> xslt_process('cim30400'::text, > >> $$http://www.w3.org/1999/XSL/Transform"; > >> version="1.0"> > >> > >> > > [snip] > >> > >> $$::text, 'n1=v

Re: [HACKERS] Initial review of xslt with no limits patch

2010-08-05 Thread Pavel Stehule
2010/8/6 David Fetter : > On Fri, Aug 06, 2010 at 05:57:37AM +0200, Pavel Stehule wrote: >> 2010/8/6 Andrew Dunstan : >> > On 08/05/2010 06:56 PM, Mike Fowler wrote: >> >> SELECT >> >> xslt_process('cim30400'::text, >> >> $$http://www.w3.org/1999/XSL/Transform"; >> >> version="1.0"> >> >> >> >> >>

<    1   2