FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fix: Save a reference to created tasks, to avoid tasks disappearing by anio · Pull Request #2005 · microsoft/botbuilder-python · GitHub

This repository was archived by the owner on Jan 5, 2026. It is now read-only.

fix: Save a reference to created tasks, to avoid tasks disappearing - #2005

Merged
tracyboehrer (tracyboehrer) merged 3 commits into
microsoft:mainfrom
anio:main
Jan 26, 2024
Merged

fix: Save a reference to created tasks, to avoid tasks disappearing#2005
tracyboehrer (tracyboehrer) merged 3 commits into
microsoft:mainfrom
anio:main

Conversation

ali (anio) commented Jun 28, 2023
edited by tracyboehrer
Loading

Copy link
Copy Markdown
Contributor

#minor

This change saves references for created tasks to avoid tasks disappearing.

Save a reference to the result of this function, to avoid a task disappearing mid-execution. The event loop only keeps weak references to tasks. A task that isn’t referenced elsewhere may get garbage collected at any time, even before it’s done.

python docs

Please review the link: https://textual.textualize.io/blog/2023/02/11/the-heisenbug-lurking-in-your-async-code/

ali (anio) requested a review from a team as a code owner June 28, 2023 12:02

Copy link
Copy Markdown
Contributor Author

@microsoft-github-policy-service agree

tracyboehrer (tracyboehrer) changed the title save a reference to created tasks, to avoid tasks disappearing fix: Save a reference to created tasks, to avoid tasks disappearing Jan 25, 2024
tracyboehrer (tracyboehrer) merged commit 786650e into microsoft:main Jan 26, 2024
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL