On 4/26/09 6:23 PM, Tom Lane wrote:
Why in the world would you confine the feature to just two data types?
This seems like a fundamentally incorrect approach.

The reason why I wrote it that way is because that's the way plpythonu's conversion to python and back is set up.

Without my patch it currently knows how to convert BOOLOID, FLOAT4OID, FLOAT8OID, NUMERICOID, INT2OID, INT4OID, INT8OID from pgsql form to python form and back again as function return types. All the rest of the pgsql types are converted into python strings with 1:1 textual representation from pgsql. (This includes all array types)

What my patch basically changes is that instead of making an INT4/TEXT array a python string, it converts them to python list()'s with corresponding types. The rest are still being converted into python strings. So I started with INT[] and TEXT[] because those were the ones I actually needed myself but extending conversion support to for example to FLOATs/NUMERICs wouldn't be that hard.

It also basically corresponds with postgresql/src/pl/plpython/TODO's:
"* Allow arrays as function arguments and return values.  (almost done)"

- Hannu


--
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