"Marko Kreen" <[EMAIL PROTECTED]> writes:
> On 11/23/07, Tom Lane <[EMAIL PROTECTED]> wrote:
>> Why? I can't imagine any real use for it. If you're thinking that
>> it could provide a guide as to what to resize the buffer to, think
>> again.
> If the output was truncated due to this limit then
On 11/23/07, Tom Lane <[EMAIL PROTECTED]> wrote:
> "Marko Kreen" <[EMAIL PROTECTED]> writes:
> > Just a note - appendStringInfoVA should take *nprinted argument.
>
> Why? I can't imagine any real use for it. If you're thinking that
> it could provide a guide as to what to resize the buffer to, th
"Marko Kreen" <[EMAIL PROTECTED]> writes:
> Just a note - appendStringInfoVA should take *nprinted argument.
Why? I can't imagine any real use for it. If you're thinking that
it could provide a guide as to what to resize the buffer to, think
again.
regards, tom lane
---
On 11/22/07, Alvaro Herrera <[EMAIL PROTECTED]> wrote:
> One problem here is that 7.3 does not have appendStringInfoVA. From 7.4
> onwards there is no problem. Should I backport the code from 7.4? (On
> a first try to do it I introduced a nasty bug, so it's not as easy as
> a cut'n paste).
Just
On 11/23/07, Alvaro Herrera <[EMAIL PROTECTED]> wrote:
> Marko Kreen escribió:
> > Problem turns out to be va_list handling in PLy_vprintf() which
> > uses same va_list repeatedly. Fix is to va_copy to temp variable.
>
> Marko, I just applied what I posted earlier today from 7.4 to 8.2 and
> HEAD.
Alvaro Herrera <[EMAIL PROTECTED]> writes:
> What I'm going to do is commit the fix to just 7.4 onwards.
Fair enough. 7.3 is on life support anyway --- I'm not in favor of
heroic efforts to port patches that far back.
regards, tom lane
---(end of
Bruce Momjian wrote:
> OK, I researched this and realized it should have been obvious to me
> when I added this code in 2006 that making the thousands separator
> always "," for a locale of "" was going to cause a problem.
>
I tested your patch and IMHO it breaks the glibc behavior. I'm providing
Marko Kreen escribió:
> Problem turns out to be va_list handling in PLy_vprintf() which
> uses same va_list repeatedly. Fix is to va_copy to temp variable.
Marko, I just applied what I posted earlier today from 7.4 to 8.2 and
HEAD. I assume you had a more complex test case; please have a try wi
Alvaro Herrera escribió:
> One problem here is that 7.3 does not have appendStringInfoVA. From 7.4
> onwards there is no problem. Should I backport the code from 7.4? (On
> a first try to do it I introduced a nasty bug, so it's not as easy as
> a cut'n paste).
No nasty bug anywhere -- just tha
Tom Lane escribió:
> Alvaro Herrera <[EMAIL PROTECTED]> writes:
> > Tom Lane escribi�:
> >> This patch isn't acceptable because va_copy() isn't portable.
> >>
> >> I'm kinda wondering why PLy_printf and the functions after it even
> >> exist. They look like rather poorly done reimplementations of
Alvaro Herrera <[EMAIL PROTECTED]> writes:
> Tom Lane escribió:
>> This patch isn't acceptable because va_copy() isn't portable.
>>
>> I'm kinda wondering why PLy_printf and the functions after it even
>> exist. They look like rather poorly done reimplementations of
>> functionality that exists e
Tom Lane escribió:
> "Marko Kreen" <[EMAIL PROTECTED]> writes:
> > Following function crashes plpython on x86-64 / gcc 4.1.2 / debian 4.0:
> > CREATE FUNCTION crashme(str_len integer)
> > RETURNS text AS $$
> > raise Exception("X" * str_len)
> > $$ LANGUAGE plpythonu;
>
> > SELECT cras
"Marko Kreen" <[EMAIL PROTECTED]> writes:
> Following function crashes plpython on x86-64 / gcc 4.1.2 / debian 4.0:
> CREATE FUNCTION crashme(str_len integer)
> RETURNS text AS $$
> raise Exception("X" * str_len)
> $$ LANGUAGE plpythonu;
> SELECT crashme(1000);
> Problem turns out to
Tom Lane napsal(a):
(a) it wasn't "to build at all", it was to allow "inline" to be enabled
on Sun Studio's compiler, which apparently is too dumb to not generate
copies of an unreferenced inline function.
I don't think that you have right. inline is just optimizer hint and Sun Studio
perfor
Here's the WIP patch I described on -hackers to implemented "ordered" append
nodes.
merge-append-v1.diff.gz
Description: Binary data
--
Gregory Stark
EnterpriseDB http://www.enterprisedb.com
Ask me about EnterpriseDB's Slony Replication support!
---(end
Following function crashes plpython on x86-64 / gcc 4.1.2 / debian 4.0:
CREATE FUNCTION crashme(str_len integer)
RETURNS text AS $$
raise Exception("X" * str_len)
$$ LANGUAGE plpythonu;
SELECT crashme(1000);
Problem turns out to be va_list handling in PLy_vprintf() which
uses same va
Small patch to reduce archive messages to DEBUG1 attached.
Forwarded Message
From: Simon Riggs <[EMAIL PROTECTED]>
To: Tom Lane <[EMAIL PROTECTED]>
Cc: Kevin Grittner <[EMAIL PROTECTED]>, Rudolf van der Leeden
<[EMAIL PROTECTED]>, [EMAIL PROTECTED]
Subject: Re: [HACKERS] Postgres
17 matches
Mail list logo
