Re: Bug with callable statement and output parameters

New patch.


On 27-Apr-06, at 10:15 PM, Kris Jurka wrote:

>
>
> On Thu, 27 Apr 2006, Dave Cramer wrote:
>
>> Patch attached for review and test
>>
>
> This patch does not handle the situation where the number of
> registered parameters does not match the number of returned
> parameters as the attached tests show.
>
> Also the regression tests you sent me separately are no good.  They
> do not pass because they mix creating and destroying the test
> function in the test method and the tearDown method.  This should
> go into setUp and tearDown.  Also you can't call executeQuery on a
> CallableStatement because it will complain with "No results were
> returned by the query."
>
> The whole Jdbc3CallableStatementTest sucks.  It should have all of
> its function/table creation destruction moved into setUp/tearDown.
> Also the coding:
>
> try {
>     // something
> } catch(Exception e) {
>     fail(e.getMessage();
> }
>
> Is a waste of code.  Not only does it not do anything, but it also
> obscures the source of the exception.
>
> Kris Jurka
> <OutFunc.java>
>
> ---------------------------(end of
> broadcast)---------------------------
> TIP 4: Have you searched our list archives?
>
>                http://archives.postgresql.org