On Sun, 2013-07-07 at 02:01 -0300, Claudio Freire wrote:
> You really want to test more than just the str. The patch contains
> casts to int and float, which is something existing PLPython code will
> be doing, so it's good to test it still can be done with decimal.

Let's remember that we are regression testing PL/Python here, not
Python.  The functionality of PL/Python is to convert a numeric to
Decimal, and that's what we are testing.  What Python can or cannot do
with that is not our job to test.

> Existing python code will also expect the number to be a float, and
> will try to operate against other floats. That's not going to work
> anymore, that has to go into release notes.

Right, this will be listed in the release notes under upgrade caveats.





-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to