Re: [PATCHES] better support of out parameters in plperl

2007-02-08 Thread Bruce Momjian
Would someone review this. It is in the patches_hold queue: http://momjian.postgresql.org/cgi-bin/pgpatches_hold --- Andrew Dunstan wrote: I think it has to wait to 8.3. It's a complete mess that was submitted

Re: [PATCHES] better support of out parameters in plperl

2007-02-08 Thread Andrew Dunstan
Has it been resubmitted with issues attended to? If it has, I missed it. If not, why should someone else waste time on something so broken that it produced a stringified perl hashref on output? It should never have gone back in the queue IMNSHO. cheers andrew Bruce Momjian wrote: Would

Re: [PATCHES] better support of out parameters in plperl

2007-02-08 Thread Bruce Momjian
Andrew Dunstan wrote: Has it been resubmitted with issues attended to? If it has, I missed it. If not, why should someone else waste time on something so broken that it produced a stringified perl hashref on output? It should never have gone back in the queue IMNSHO. Fine, removed. I

Re: [PATCHES] better support of out parameters in plperl

2007-02-08 Thread Bruce Momjian
This patch has been rejected based on comments just made by Andrew Dunstan. If the author wants to revisit that, please reply and we can discuss the issues. --- Andrew Dunstan wrote: I wrote: Pavel Stehule wrote:

Re: [PATCHES] better support of out parameters in plperl

2006-09-02 Thread Bruce Momjian
Uh, were are we in fixing/reviewing this? --- Andrew Dunstan wrote: I wrote: Pavel Stehule wrote: Hello, I send two small patches. First does conversion from perl to postgresql array in OUT parameters.

Re: [PATCHES] better support of out parameters in plperl

2006-09-02 Thread Andrew Dunstan
I think it has to wait to 8.3. It's a complete mess that was submitted unheralded at the last moment. Pavel needs to get into the habit of submitting ideas first, not just patches. And there must be proper documentation and working regression tests. cheers andrew Bruce Momjian wrote: Uh,

Re: [PATCHES] better support of out parameters in plperl

2006-09-02 Thread Bruce Momjian
OK. This has been saved for the 8.3 release: http://momjian.postgresql.org/cgi-bin/pgpatches_hold --- Andrew Dunstan wrote: I think it has to wait to 8.3. It's a complete mess that was submitted unheralded at

Re: [PATCHES] better support of out parameters in plperl

2006-09-02 Thread Bruce Momjian
Oh, let me add that this was first discussed on July 28: http://archives.postgresql.org/pgsql-hackers/2006-07/msg01421.php and a patch posted on July 30: http://archives.postgresql.org/pgsql-hackers/2006-07/msg01559.php

Re: [PATCHES] better support of out parameters in plperl

2006-09-02 Thread Andrew Dunstan
Like I said, at the last moment. Bruce Momjian wrote: Oh, let me add that this was first discussed on July 28: http://archives.postgresql.org/pgsql-hackers/2006-07/msg01421.php and a patch posted on July 30: http://archives.postgresql.org/pgsql-hackers/2006-07/msg01559.php

Re: [PATCHES] better support of out parameters in plperl

2006-09-02 Thread Tom Lane
Bruce Momjian [EMAIL PROTECTED] writes: Uh, were are we in fixing/reviewing this? It's dead for 8.2 --- Andrew's complaints are pretty serious at both the conceptual and implementation levels, and there's been no sign of discussion about how to fix them. regards, tom

Re: [PATCHES] better support of out parameters in plperl

2006-08-13 Thread Andrew Dunstan
I wrote: Pavel Stehule wrote: Hello, I send two small patches. First does conversion from perl to postgresql array in OUT parameters. Second patch allow hash form output from procedures with one OUT argument. I will try to review these in the next 2 weeks unless someone beats me to

Re: [PATCHES] better support of out parameters in plperl

2006-08-13 Thread Bruce Momjian
Based on this analysis, and problems with differing regression results on different platforms, this attached patch has been reverted. --- Andrew Dunstan wrote: I wrote: Pavel Stehule wrote: Hello, I send two

Re: [PATCHES] better support of out parameters in plperl

2006-08-11 Thread Bruce Momjian
Patch applied. Thanks. --- Pavel Stehule wrote: Hello, I send two small patches. First does conversion from perl to postgresql array in OUT parameters. Second patch allow hash form output from procedures with one

Re: [PATCHES] better support of out parameters in plperl

2006-08-01 Thread Andrew Dunstan
Pavel Stehule wrote: Hello, I send two small patches. First does conversion from perl to postgresql array in OUT parameters. Second patch allow hash form output from procedures with one OUT argument. I will try to review these in the next 2 weeks unless someone beats me to it. cheers