From: | Priyanka Shendge <priyanka(dot)shendge(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: Synonym node for EPAS (pgAdmin4) |
Date: | 2016-08-26 11:28:47 |
Message-ID: | CAKmZXFQBrcVSSWtKCekB07_hdXOQSmUp6GVN0nR6g1pN7g1-kA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
1 more point to add:
5. Synonyms should only run for EPAS server
On 26 August 2016 at 16:44, Priyanka Shendge <
priyanka(dot)shendge(at)enterprisedb(dot)com> wrote:
> Hi Murtuza,
>
> Please find my review comments below:
>
> 1. Please attach test_advanced_config.json.in file to read the config data
> 2. While reading data from config file "owner" field is missing
> 3. Please add synonym pickle object to ../regression/test_utils.py file
> 4. Can we have config data parameters generated dynamically? (e.g. schema
> name)
>
> Let me know for any assistance.
>
> Thank you.
>
> On 26 August 2016 at 13:03, Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
> wrote:
>
>> Thanks, patch applied.
>>
>> On Thu, Aug 25, 2016 at 7:58 PM, Murtuza Zabuawala <
>> murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
>>
>>> Hi,
>>>
>>> Please find updated patch.
>>>
>>> I have also fixed the issue in package node which was displaying create
>>> menu in PG server. (separate patch attached)
>>>
>>> --
>>> Regards,
>>> Murtuza Zabuawala
>>> EnterpriseDB: http://www.enterprisedb.com
>>> The Enterprise PostgreSQL Company
>>>
>>> On Thu, Aug 25, 2016 at 4:34 PM, Akshay Joshi <
>>> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>>>
>>>> Hi Murtuza
>>>>
>>>> Following are my review comments:
>>>>
>>>> - Create "Synonym" option is visible for Postgres Servers, when
>>>> right click on schema node.
>>>>
>>>> Fixed
>>>
>>>>
>>>> - Unable to create Public Synonym faced "Internal Server Error".
>>>>
>>>> Fixed
>>>
>>>>
>>>> - SQL help not working when click on "[i]" button.
>>>>
>>>> Fixed, Now it is disbaled
>>>
>>>>
>>>> - "Public Synonyms?" switch control showing True/False when click
>>>> on Synonyms collection node and it is showing Yes/No when click on synonym
>>>> node under properties tab.
>>>>
>>>> Fixed
>>>
>>>>
>>>> On Thu, Aug 25, 2016 at 3:08 PM, Murtuza Zabuawala <
>>>> murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
>>>>
>>>>> Hi,
>>>>>
>>>>> PFA updated patch with `--binary` option for .png files.
>>>>>
>>>>> --
>>>>> Regards,
>>>>> Murtuza Zabuawala
>>>>> EnterpriseDB: http://www.enterprisedb.com
>>>>> The Enterprise PostgreSQL Company
>>>>>
>>>>> On Thu, Aug 25, 2016 at 12:27 PM, Murtuza Zabuawala <
>>>>> murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
>>>>>
>>>>>> Hi,
>>>>>>
>>>>>> PFA patch for synonym node, this patch also includes basic unit test
>>>>>> cases.
>>>>>> Please review.
>>>>>>
>>>>>> --
>>>>>> 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
>>>>>
>>>>>
>>>>
>>>>
>>>> --
>>>> *Akshay Joshi*
>>>> *Principal Software Engineer *
>>>>
>>>>
>>>>
>>>> *Phone: +91 20-3058-9517Mobile: +91 976-788-8246*
>>>>
>>>
>>>
>>
>>
>> --
>> *Akshay Joshi*
>> *Principal Software Engineer *
>>
>>
>>
>> *Phone: +91 20-3058-9517Mobile: +91 976-788-8246*
>>
>
>
>
> --
> Best,
> Priyanka
>
> EnterpriseDB Corporation
> The Enterprise PostgreSQL Company
>
--
Best,
Priyanka
EnterpriseDB Corporation
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Neel Patel | 2016-08-26 11:31:44 | [pgAdmin4][patch]: RM #1226 |
Previous Message | Priyanka Shendge | 2016-08-26 11:14:57 | Re: PATCH: Synonym node for EPAS (pgAdmin4) |