From: | Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com> |
---|---|
To: | Aditya Toshniwal <aditya(dot)toshniwal(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [pgAdmin][RM4590] Schema Backup fails with names that requires quoting |
Date: | 2019-10-10 12:30:30 |
Message-ID: | CANxoLDdbyQvNpXVu-7U2_fRmT=eOynM7_2UqtoPaJyuyNQf0oQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Thanks, patch applied.
On Wed, Oct 9, 2019 at 2:21 PM Aditya Toshniwal <
aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:
> Hi Hackers,
>
> Attached is the updated patch. Tested on MacOS, Windows 7, Ubuntu 16 and
> works fine.
> Kindly review.
>
> On Wed, Oct 9, 2019 at 1:48 PM Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
> wrote:
>
>> Hi Aditya
>>
>> I have tried your patch on MAC. Backup job is completed successfully but
>> got the following message:
>> pg_dump: last built-in OID is 16383
>> pg_dump: no matching schemas were found
>>
>> Please verify it once again and send the modified patch.
>>
>> On Tue, Oct 8, 2019 at 5:46 PM Aditya Toshniwal <
>> aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:
>>
>>> Hi Hackers,
>>>
>>> Turned out to be an issue only on Mac. Backup works fine for other OS.
>>> Attached is the patch to fix the issue on Mac. Code is changed to add
>>> extra pairs of quotes wherever required on MacOS.
>>>
>>> Kindly review.
>>>
>>> On Thu, Oct 3, 2019 at 3:53 PM Aditya Toshniwal <
>>> aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:
>>>
>>>> Hi Hackers,
>>>>
>>>> Attached is the patch to fix the schema backup with names that requires
>>>> double quotes.
>>>> Command line has special meaning for double quotes and so the command
>>>> is run without the double quotes if applied on objects. Code is changed to
>>>> add extra pairs of quotes to save our object quotes.
>>>>
>>>> Also fixed a test case which was failing if certain paths had spaces in
>>>> between.
>>>>
>>>> Kindly review.
>>>>
>>>> --
>>>> Thanks and Regards,
>>>> Aditya Toshniwal
>>>> Sr. Software Engineer | EnterpriseDB India | Pune
>>>> "Don't Complain about Heat, Plant a TREE"
>>>>
>>>
>>>
>>> --
>>> Thanks and Regards,
>>> Aditya Toshniwal
>>> Sr. Software Engineer | EnterpriseDB India | Pune
>>> "Don't Complain about Heat, Plant a TREE"
>>>
>>
>>
>> --
>> *Thanks & Regards*
>> *Akshay Joshi*
>>
>> *Sr. Software Architect*
>> *EnterpriseDB Software India Private Limited*
>> *Mobile: +91 976-788-8246*
>>
>
>
> --
> Thanks and Regards,
> Aditya Toshniwal
> Sr. Software Engineer | EnterpriseDB India | Pune
> "Don't Complain about Heat, Plant a TREE"
>
--
*Thanks & Regards*
*Akshay Joshi*
*Sr. Software Architect*
*EnterpriseDB Software India Private Limited*
*Mobile: +91 976-788-8246*
From | Date | Subject | |
---|---|---|---|
Next Message | Akshay Joshi | 2019-10-10 13:25:30 | pgAdmin 4 commit: Fix issue where VALID foreign keys show as NOT VALID |
Previous Message | Akshay Joshi | 2019-10-10 12:30:09 | pgAdmin 4 commit: popen() function strips the quotes from the arguments |