Re: PATCH: Improvement in the debugger support for EnterpriseDB <= 9.2

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: PATCH: Improvement in the debugger support for EnterpriseDB <= 9.2
Date: 2013-10-23 15:23:49
Message-ID: CA+OCxoyoDi3B74xKLS4cbCHu=oe91e6ytm1kerCoX0n301ETzA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi

On Wed, Oct 23, 2013 at 8:09 AM, Ashesh Vashi
<ashesh(dot)vashi(at)enterprisedb(dot)com> wrote:
> Hi All,
>
> While testing the debugger at EnterpriseDB with PPAS/PostgreSQL, we found
> some issues as below:
> 1. Function/Procedure execution completed with an error - "no target for
> SPL-style OUT parameter" with (PPAS <= 9.3).
> Resolution:
> Call the functions/procedure with the value explicitly (instead of using the
> PQexecParams - libpq API and also, edb-spl language needs to be called from
> an anonymous block) and fetch the result properly for PPAS <= 9.2.
>
> 2. Variadic function with the OUT parameter is not debuggable on
> PostgreSQL/PPAS.
> Resolution:
> Fix the IN/INOUT parameters numbering properly, before creating the function
> properly.
>
> 3. On execution completion an edb-spl function is not showing return value
> with PPAS
> Resolution:
> As we can not return a result from the anonymous block, we needs to print it
> from the anonymous block. Hence - those results will be printed in the
> messages window instead of result window for them.
>
> 4. Showing the current execution marker at wrong location (offset to 1) for
> PPAS <= 9.0
> Resolution:
> Find out the execution marker position based on the offset based on each
> server version.
>
> Attached is the patch, which takes care of all of above issues.
>
> Please review it.
> If nobody will have the objection, I will check-in the patch in
> REL-1_18_PATCHES and main branches.

I've eyeballed it and don't see anything objectionable. As you wrote
the debugger though, you're in a far better position to say if it's
done the right way though :-)

Can you get one of the EDB team to test it?

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Ashesh Vashi 2013-10-23 16:52:07 Re: PATCH: Improvement in the debugger support for EnterpriseDB <= 9.2
Previous Message Sameer Kumar 2013-10-23 14:42:04 Re: Using Index to do sorting while using windowing function