Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

[3.8] bpo-41247: asyncio.set_running_loop() cache running loop holder (GH-21401)#21403

Closed
miss-islington wants to merge 1 commit into
python:3.8python/cpython:3.8from
miss-islington:backport-529f426-3.8miss-islington/cpython:backport-529f426-3.8Copy head branch name to clipboard
Closed

[3.8] bpo-41247: asyncio.set_running_loop() cache running loop holder (GH-21401)#21403
miss-islington wants to merge 1 commit into
python:3.8python/cpython:3.8from
miss-islington:backport-529f426-3.8miss-islington/cpython:backport-529f426-3.8Copy head branch name to clipboard

Conversation

@miss-islington

@miss-islington miss-islington commented Jul 8, 2020

Copy link
Copy Markdown
Contributor

The running loop holder cache variable was always set to NULL when
calling set_running_loop.

Now set_running_loop saves the newly created running loop holder in the
cache variable for faster access in get_running_loop.

Automerge-Triggered-By: @1st1
(cherry picked from commit 529f426)

Co-authored-by: Tony Solomonik tony.solomonik@gmail.com

https://bugs.python.org/issue41247

Automerge-Triggered-By: @1st1

…onGH-21401)

The running loop holder cache variable was always set to NULL when
calling set_running_loop.

Now set_running_loop saves the newly created running loop holder in the
cache variable for faster access in get_running_loop.

Automerge-Triggered-By: @1st1
(cherry picked from commit 529f426)

Co-authored-by: Tony Solomonik <tony.solomonik@gmail.com>
@miss-islington

Copy link
Copy Markdown
Contributor Author

@tontinton: Status check is done, and it's a failure ❌ .

@tontinton

tontinton commented Jul 8, 2020

Copy link
Copy Markdown
Contributor

@tontinton: Status check is done, and it's a failure ❌ .

I've created a branch with the same fix only on 3.8, but github has issues right now, so I'll update when the PR is up, I will probably edit this comment with the PR link or something

PR: #21406

@1st1 1st1 closed this Jul 8, 2020
@1st1 1st1 reopened this Jul 8, 2020
@miss-islington

Copy link
Copy Markdown
Contributor Author

@tontinton: Status check is done, and it's a failure ❌ .

2 similar comments
@miss-islington

Copy link
Copy Markdown
Contributor Author

@tontinton: Status check is done, and it's a failure ❌ .

@miss-islington

Copy link
Copy Markdown
Contributor Author

@tontinton: Status check is done, and it's a failure ❌ .

@miss-islington miss-islington deleted the backport-529f426-3.8 branch July 8, 2020 23:01
@miss-islington

Copy link
Copy Markdown
Contributor Author

@tontinton: Status check is done, and it's a failure ❌ .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.