Thanks to visit codestin.com
Credit goes to github.com

Skip to content

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

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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed

Conversation

miss-islington
Copy link
Contributor

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

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 [email protected]

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 <[email protected]>
@miss-islington
Copy link
Contributor Author

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

@tontinton
Copy link
Contributor

tontinton commented Jul 8, 2020

@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
Contributor Author

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

2 similar comments
@miss-islington
Copy link
Contributor Author

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

@miss-islington
Copy link
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
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