Re: [HACKERS] pgsql: Fix for plpython functions; return true/false for boolean,

From: Guido Goldstein <guido(dot)goldstein(at)a-nugget(dot)org>
To: Peter Eisentraut <peter_e(at)gmx(dot)net>
Cc: pgsql-hackers(at)postgresql(dot)org, pgsql-patches(at)postgresql(dot)org
Subject: Re: [HACKERS] pgsql: Fix for plpython functions; return true/false for boolean,
Date: 2007-01-30 13:52:56
Message-ID: 45BF4DB8.7060909@a-nugget.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers pgsql-patches

Peter Eisentraut wrote:
> Guido Goldstein wrote:
>> Is it possible to tell me which python versions you want to
>> support?
>
> The issue isn't so much which versions we want to support. There is
> certainly some flexibility with that. But when a patch breaks the
> buildfarm a) unannounced and b) without any apparent feature gain, then
> people get annoyed.

If this breaks the buildfarm it's not my failure.
Except you can tell me what I've got to do with the
buildfarm.

If you mean that plpython didn't compile, fine; simply tell
the people what version they should consider when sending
in patches.

I've checked the patch with postgres 8.1.3 and 8.2.1
with python 2.4 and 2.5 on intel 32 bit and amd 64 bit
systems; all systems running linux.

*And* it's not a feature patch but a bug-fixing one!
Python is a language with strong typing, so silently
converting a datatype is a bug -- not a feature.
Btw, you'll lose the type information of boolean columns in
trigger functions (NEW and OLD dicts, no explicit parameters),
which does cause problems.

> That said, we certainly try to support a few more versions of Python
[...]

If you want to support python 2.3 use the attached patch, which also
works for the newer python versions.
The Python 2.3 branch is the oldest _officially_ supported python version.

Anyway, to circumvent the above mentiond point a) I herewith anncounce
that the included patch might break the buildfarm.

Cheers
Guido

Attachment Content-Type Size
pg-plpython.diff text/x-patch 396 bytes

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Andrew Dunstan 2007-01-30 14:24:22 Re: pgsql: Fix for plpython functions; return true/false for boolean,
Previous Message Peter Eisentraut 2007-01-30 12:39:19 Re: pgsql: Fix for plpython functions; return true/false for boolean,

Browse pgsql-hackers by date

  From Date Subject
Next Message Andrew Dunstan 2007-01-30 14:24:22 Re: pgsql: Fix for plpython functions; return true/false for boolean,
Previous Message Peter Eisentraut 2007-01-30 12:39:19 Re: pgsql: Fix for plpython functions; return true/false for boolean,

Browse pgsql-patches by date

  From Date Subject
Next Message Andrew Dunstan 2007-01-30 14:24:22 Re: pgsql: Fix for plpython functions; return true/false for boolean,
Previous Message Peter Eisentraut 2007-01-30 12:39:19 Re: pgsql: Fix for plpython functions; return true/false for boolean,