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 12:11:12 |
Message-ID: | CAKKotZSANwaaaXWa-nu2GbZt7buQ4PdGduht=Bc6yX3NzNB1GA@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 with minor enhancements in code.
Regards,
Murtuza
--
Regards,
Murtuza Zabuawala
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
On Tue, Feb 23, 2016 at 1:32 PM, Murtuza Zabuawala <
murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
> 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_v5.patch | application/octet-stream | 181.5 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Khushboo Vashi | 2016-02-23 13:23:38 | [pgAdmin4] [Patch]: Foreign Table Module |
Previous Message | Surinder Kumar | 2016-02-23 11:07:22 | Re: [pgAdmin4] [Patch]: Extension Module |