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: Tablespace Node [pgAdmin4] |
Date: | 2016-02-03 10:11:52 |
Message-ID: | CAKKotZQHy+cg2Ph8bvrVpn=M7G=4-dGB-55dpniiA2+ourvx8A@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi Neel,
Thanks for reviewing my code, I have fixed the issues mentioned,
Please find attached updated patch for table space node.
--
Regards,
Murtuza Zabuawala
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
On Wed, Feb 3, 2016 at 2:42 PM, Neel Patel <neel(dot)patel(at)enterprisedb(dot)com>
wrote:
> Hi Murtuza,
>
> Please find below review comments.
>
> - "Location" and "Variables" fields are not getting display under
> Properties Panel.
> - When we remove/add the Variables fields, we are executing query
> individually for each parameters. We should execute it in single query as
> below.
>
> e.g.
>
> *Multiple Query :- *
> ALTER TABLESPACE "1 tb"
> SET (random_page_cost=123);
> ALTER TABLESPACE "1 tb"
> SET (seq_page_cost=345);
>
> *Single Query :- *
> ALTER TABLESPACE "1 tb"
> SET (random_page_cost=123, seq_page_cost=345);
>
>
> * Multiple Query :- *
> ALTER TABLESPACE "1 tb"
> RESET (random_page_cost);
> ALTER TABLESPACE "1 tb"
> RESET (seq_page_cost);
>
> * Single Query :- *
> ALTER TABLESPACE "1 tb"
> RESET (random_page_cost, seq_page_cost);
>
> Other functionalities are working fine. Let us know in case of any issues.
>
> Thanks,
> Neel Patel
>
> On Wed, Feb 3, 2016 at 12:57 PM, Murtuza Zabuawala <
> murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
>
>> Hi,
>>
>> PFA updated patch, Copyright year updated.
>>
>>
>> --
>> Regards,
>> Murtuza Zabuawala
>> EnterpriseDB: http://www.enterprisedb.com
>> The Enterprise PostgreSQL Company
>>
>> On Wed, Feb 3, 2016 at 12:22 PM, Murtuza Zabuawala <
>> murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
>>
>>> Hi,
>>>
>>> Please find attach patch for tablespace node for pgAdmin4.
>>>
>>>
>>> --
>>> 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 |
---|---|---|
tablespace_node_v3.patch | application/octet-stream | 44.2 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Khushboo Vashi | 2016-02-03 10:27:29 | Re: PATCH: NodeAjaxOptionsControl in Subnode |
Previous Message | Murtuza Zabuawala | 2016-02-03 10:07:35 | Re: Added variable macro for generic db objects |