From: | Khushboo Vashi <khushboo(dot)vashi(at)enterprisedb(dot)com> |
---|---|
To: | Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [pgAdmin 4][Patch]: #7314 - psycopg2 to psycopg3 migration |
Date: | 2022-09-09 05:31:47 |
Message-ID: | CAFOhELeDMsS3_hVP4yy3Z6UVzU7HsFOg+Pt2w-LUKrrdBG29fw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hello,
Please find the attached updated patch.
On Thu, Sep 1, 2022 at 1:01 PM Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
wrote:
> Hi Khushboo
>
> Following are the review comments:
>
> - Remove psycopg2 from requirements.txt
>
> Removed, the code also removed.
>
> - Set PG_DEFAULT_DRIVER to psycopg3 in config.py
>
> Done.
>
> - Remove all the imports of 'psycopg2' from the entire project.
>
> Done.
>
> - Query Tool not working with psycopg3 == 3.1
>
> Done.
Thanks,
Khushboo
>
> On Tue, Aug 30, 2022 at 8:57 PM Khushboo Vashi <
> khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:
>
>> Hello,
>>
>> Please find the attached patch to migrate psycopg2 to psycopg3. I have
>> used psycopg v 3.1.0 which is yet to be released soon. To test the patch,
>> one needs to clone the psycopg3 repo, build it and point it to the pgadmin
>> venv.
>>
>> For the time being, psycopg2 and psycopg3 both can be configurable via PG_DEFAULT_DRIVER
>> and can be used.
>>
>> With this patch many SQL Ascii related issues and Range partition
>> related issues are resolved. For example: #7028, # 6575, #4061, #7165.
>>
>> I have verified some of them but we need to test all of them.
>>
>> Note: Debugger test cases are still failing, on which I am still working.
>>
>> Thanks,
>> Khushboo
>>
>
>
> --
>
> <http://www.enterprisedb.com>
>
> Akshay Joshi
>
> Principal Software Architect
>
> +91 9767888246
>
> www.enterprisedb.com
>
> <https://www.linkedin.com/company/edbpostgres>
> <https://twitter.com/edbpostgres?lang=en>
> <https://www.facebook.com/EDBpostgres>
> <https://www.instagram.com/EDBpostgres/>
>
Attachment | Content-Type | Size |
---|---|---|
psycopg3_migration_v1.patch | application/octet-stream | 824.7 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Khushboo Vashi | 2022-09-09 05:42:40 | Re: Housekeeping #7446 [docs] Update keyboard shortcut docs page |
Previous Message | Akshay Joshi | 2022-09-08 14:58:50 | pgAdmin 4 commit: Fixed 'Remove the unnecessary boolean literals' code |