Re: [pgAdmin 4][Patch]: #7314 - psycopg2 to psycopg3 migration - Mailing list pgadmin-hackers

From Khushboo Vashi
Subject Re: [pgAdmin 4][Patch]: #7314 - psycopg2 to psycopg3 migration
Date
Msg-id CAFOhELeDMsS3_hVP4yy3Z6UVzU7HsFOg+Pt2w-LUKrrdBG29fw@mail.gmail.com
Whole thread Raw
In response to Re: [pgAdmin 4][Patch]: #7314 - psycopg2 to psycopg3 migration  (Akshay Joshi <akshay.joshi@enterprisedb.com>)
Responses Re: [pgAdmin 4][Patch]: #7314 - psycopg2 to psycopg3 migration  (Khushboo Vashi <khushboo.vashi@enterprisedb.com>)
List pgadmin-hackers
Hello,

Please find the attached updated patch.

On Thu, Sep 1, 2022 at 1:01 PM Akshay Joshi <akshay.joshi@enterprisedb.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.vashi@enterprisedb.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


--

Akshay Joshi

Principal Software Architect

+91 9767888246

www.enterprisedb.com

     

Attachment

pgadmin-hackers by date:

Previous
From: Akshay Joshi
Date:
Subject: pgAdmin 4 commit: Fixed 'Remove the unnecessary boolean literals' code
Next
From: Khushboo Vashi
Date:
Subject: Re: Housekeeping #7446 [docs] Update keyboard shortcut docs page