From: | Neel Patel <neel(dot)patel(at)enterprisedb(dot)com> |
---|---|
To: | Murtuza Zabuawala <murtuza(dot)zabuawala(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: PATCH: Schema/Catalog Node [pgAdmin4] |
Date: | 2016-02-11 06:28:16 |
Message-ID: | CACCA4P0qfNOr97UXWm=LnKfbdcsiS_OZ-nOhseBMSH0TQagyRA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
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
>
>
From | Date | Subject | |
---|---|---|---|
Next Message | Akshay Joshi | 2016-02-11 06:39:27 | Re: patch for cast module |
Previous Message | Harshal Dhumal | 2016-02-11 06:26:40 | Privilege control: disable with grant ops if grantee is public [pgAdmin4] |