I applied the patch to 7.4.1 on RH Linux 8.0 and it works great.



-----Original Message-----
From: Joe Conway [mailto:[EMAIL PROTECTED] 
Sent: Thursday, March 04, 2004 11:12 PM
Cc: Tom Lane; Oleg Lebedev
Subject: Re: [PATCHES] [GENERAL] dblink: rollback transaction

Joe Conway wrote:
> Tom Lane wrote:
>> Joe Conway <[EMAIL PROTECTED]> writes:
>>> I like the idea in general, but maybe instead there should be a new
>>> overloaded version of the existing function names that accepts an 
>>> additional bool argument. Without the argument, behavior would be as

>>> it is now; with it, you could specify the old or new behavior.
>> Um, maybe I'm confused about the context, but aren't we talking about

>> C function names here?  No overloading is possible in C ...
> I was thinking in terms of overloaded SQL function names. For example,
> in addition to dblink_exec(text) and dblink_exec(text,text) we create 
> dblink_exec(text,bool) and dblink_exec(text,text,bool).
> Currently both SQL versions of dblink_exec are implemented by a single

> C
> level function. But yes, we'd need another C level function to support

> the new SQL functions because there would be no way to distinguish the
> two-argument versions otherwise. (Actually, now I'm wondering if we 
> could use a single C function for all four SQL versions -- between 
> PG_NARGS() and get_fn_expr_argtype() we should be able to figure out
> we were called, shouldn't we?)

The attached implements the new overloaded SQL functions as discussed 
above (i.e. start with existing argument combinations, add a new bool 
argument to each). I ended up with a single C function (by making use of

number and type of the arguments) for each overloaded SQL function name.

I'll commit in a day or two if there are no objections.


This e-mail may contain privileged or confidential material intended for the named 
recipient only. 
If you are not the named recipient, delete this message and all attachments. 
Unauthorized reviewing, copying, printing, disclosing, or otherwise using information 
in this e-mail is prohibited. 
We reserve the right to monitor e-mail sent through our network.  

---------------------------(end of broadcast)---------------------------
TIP 8: explain analyze is your friend

Reply via email to