NotificationsYou must be signed in to change notification settings
Fork33.4k
Star69.8k

bpo-42064: Adapt`sqlite3` to multi-phase init (PEP 489)#29234

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Merged

encukou merged 3 commits intopython:mainfromerlend-aasland:sqlite-multiphase-init

Nov 2, 2021

Merged

bpo-42064: Adapt`sqlite3` to multi-phase init (PEP 489)#29234

encukou merged 3 commits intopython:mainfromerlend-aasland:sqlite-multiphase-init

Nov 2, 2021

Conversation

Copy link

Contributor

erlend-aasland commentedOct 27, 2021•
edited by bedevere-bot
Loading

https://bugs.python.org/issue42064

Erlend E. Aasland added2 commits

October 27, 2021 13:47

bpo-42064: Implement multiphase init

110b012

Add NEWS item

09dabdf

erlend-aasland requested a review fromberkerpeksag as acode owner

October 27, 2021 11:53

the-knights-who-say-ni added the CLA signed label

Oct 27, 2021

bedevere-bot added the awaiting review label

Oct 27, 2021

erlend-aasland requested a review fromencukou

October 27, 2021 11:53

Copy link

ContributorAuthor

erlend-aasland commentedOct 27, 2021•
edited
Loading

$ cat measure.pyimport gcimport sysfor _ in range(5):    gc.collect()    before = sys.gettotalrefcount()    import sqlite3    del sys.modules['sqlite3']    del sqlite3    gc.collect()    after = sys.gettotalrefcount()    print(f"{before=}, {after=}")$ ./python.exe measure.pybefore=51469, after=66883before=66892, after=66892before=66892, after=66892before=66892, after=66892before=66892, after=66892

Thank you so much for helping reviewing and improving this,@corona10,@vstinner,@encukou!

For a list of PR's:
$ git log --grep "bpo-$42064\|41861\|40956\|42862$" --oneline