Re: PATCH: Schema/Catalog Node [pgAdmin4]

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-23 08:02:08
Message-ID: CAKKotZSgrPU4p-SsNODeoZd09LEMb7fe5JDNvtbWEqP+H-mvmQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi All,

PFA updated patch for schema/Catalog node which includes
dependent/dependancy route handling code.

Reagrds,
Murtuza

--
Regards,
Murtuza Zabuawala
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

On Thu, Feb 11, 2016 at 4:16 PM, Murtuza Zabuawala <
murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:

> 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_v4.patch application/octet-stream 180.8 KB

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2016-02-23 09:08:57 pgAdmin 4 commit: Improve Resource Group labels per discussion with Jas
Previous Message Neel Patel 2016-02-23 07:51:22 Fixed runtime compilation error for Qt4