Skip to content
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

BUG: fix asyncio.Queue error in benchmark #2113

Merged
merged 1 commit into from
Aug 19, 2024

Conversation

Dawnfz-Lenfeng
Copy link
Contributor

  • fix benchmark异步并行测试报错 #2111
  • Previously, asyncio.Queue was created in the __init__ method rather than in the run() function. In lower versions of Python (e.g., 3.9), this could cause the queue to be attached to an incorrect event loop.
  • To address this issue, the creation of the queue has been moved to the warm_up method.
  • Additionally, a minor error in a log message has been corrected.

@XprobeBot XprobeBot added the bug Something isn't working label Aug 19, 2024
@XprobeBot XprobeBot added this to the v0.14 milestone Aug 19, 2024
Copy link
Contributor

@qinxuye qinxuye left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@qinxuye qinxuye merged commit 05b6180 into xorbitsai:main Aug 19, 2024
13 checks passed
@Dawnfz-Lenfeng Dawnfz-Lenfeng deleted the fix_queue branch September 11, 2024 07:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

benchmark异步并行测试报错
3 participants