Re: [pgadmin-hackers] PATCH: To fix the issue displaying composite result types in debugger (pgAdmin4)

2016-09-20 Thread Dave Page
Thanks - applied.

On Tue, Sep 20, 2016 at 10:46 AM, Murtuza Zabuawala
 wrote:
> Hi,
>
> PFA patch to fix the issue in debugger where we were not displaying result
> properly in result tab for composite types.
> RM#1662
>
> @Neel,
> Would you please review the patch?
>
> Issue:
> Column name key was hardcoded in dict causing overwriting other columns.
>
> --
> Regards,
> Murtuza Zabuawala
> EnterpriseDB: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
>
> --
> Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgadmin-hackers
>



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

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


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


Re: [pgadmin-hackers] PATCH: To fix the issue displaying composite result types in debugger (pgAdmin4)

2016-09-20 Thread Neel Patel
Hi Murtuza,

Sure. I will review it.

Thanks,
Neel Patel

On Tue, Sep 20, 2016 at 3:16 PM, Murtuza Zabuawala <
murtuza.zabuaw...@enterprisedb.com> wrote:

> Hi,
>
> PFA patch to fix the issue in debugger where we were not displaying result
> properly in result tab for composite types.
> RM#1662
>
> @Neel,
> Would you please review the patch?
>
> *Issue:*
> Column name key was hardcoded in dict causing overwriting other columns.
>
> --
> Regards,
> Murtuza Zabuawala
> EnterpriseDB: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
>
> --
> Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgadmin-hackers
>
>