From: | Murtuza Zabuawala <murtuza(dot)zabuawala(at)enterprisedb(dot)com> |
---|---|
To: | Neel Patel <neel(dot)patel(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: PATCH: Schema/Catalog Node [pgAdmin4] |
Date: | 2016-02-11 10:46:19 |
Message-ID: | CAKKotZQ7oujTMnaCS_LzG-RdPhiWeQkVg2Wh=O0D+fkQkEHoBg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi,
Thanks Neel for reviewing.
PFA updated patch which will fix mentioned issues.
Regards,
Murtuza
--
Regards,
Murtuza Zabuawala
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
On Thu, Feb 11, 2016 at 11:58 AM, Neel Patel <neel(dot)patel(at)enterprisedb(dot)com>
wrote:
> Hi Murtuza,
>
> Please find below review comments.
>
> - Remove the whitespace from the patch file as we are getting below
> warnings while applying the patch file.
>
> schema_catalog_node_v1.patch:1500: trailing whitespace.
> FROM pg_namespace nsp
> schema_catalog_node_v1.patch:1501: trailing whitespace.
> LEFT OUTER JOIN
> schema_catalog_node_v1.patch:1672: trailing whitespace.
> FROM pg_namespace nsp
> schema_catalog_node_v1.patch:1673: trailing whitespace.
> LEFT OUTER JOIN
> schema_catalog_node_v1.patch:2353: trailing whitespace.
> (SELECT defaclobjtype, aclexplode(defaclacl) as acl
> warning: squelched 7 whitespace errors
> warning: 12 lines add whitespace errors.
>
> - Some of the Properties are not getting displayed under Properties panel
> of schema. Make sure to display all the properties displayed in pgadmin3.
> e.g. System Schema
> - We are getting error at python side saying *"ValueError: View function
> did not return a response"* while editing the schema node.
> - Default Privileges are not getting displayed under SQL panel.
> - When we connect to PostgreSQL 9.1 database then we are getting error
> fail to execute query as below. Below error will be for both schema and
> catalog node.
>
> ERROR pgadmin: Failed to execute query (execute_2darray) for the
> server #3 - DB:postgres (Query-id: 1019298):
> Error Message:relation "pg_shseclabel" does not exist
> LINE 11: ... (SELECT array_agg(provider || '=' || label) FROM
> pg_shsecla...
>
> Thanks,
> Neel Patel
>
> On Mon, Feb 8, 2016 at 2:25 PM, Murtuza Zabuawala <
> murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
>
>> Hi,
>>
>> PFA patch for schema/catalog node.
>>
>>
>> --
>> Regards,
>> Murtuza Zabuawala
>> EnterpriseDB: http://www.enterprisedb.com
>> The Enterprise PostgreSQL Company
>>
>>
>> --
>> Sent via pgadmin-hackers mailing list (pgadmin-hackers(at)postgresql(dot)org)
>> To make changes to your subscription:
>> http://www.postgresql.org/mailpref/pgadmin-hackers
>>
>>
>
Attachment | Content-Type | Size |
---|---|---|
schema_catalog_node_v2.patch | application/octet-stream | 170.5 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Harshal Dhumal | 2016-02-11 13:14:42 | backform.pgadmin: fix for showing multiple errors for radio control [pgadmin4] |
Previous Message | Dave Page | 2016-02-11 10:39:05 | Re: [PATCH] Fix of elapsed time string generation in query window |