-
Notifications
You must be signed in to change notification settings - Fork 8.7k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
fix: apply gevent threading patch early and ensure unique workflow node execution IDs #12196
fix: apply gevent threading patch early and ensure unique workflow node execution IDs #12196
Conversation
Forcibly applying gevent theading patch in all cases may cause the following consequences:
Consider to ignore or check these situations please. |
… node execution Signed-off-by: -LAN- <laipz8200@outlook.com>
… patching directly in app.py Signed-off-by: -LAN- <laipz8200@outlook.com>
Signed-off-by: -LAN- <laipz8200@outlook.com>
6000a46
to
d66a073
Compare
Thanks for the suggestions, but I found we can only use the Gevent worker for now. |
…pp.py Signed-off-by: -LAN- <laipz8200@outlook.com>
Alright. It's just a minor reminder for you, not a blocker issue anyway. |
…de execution IDs (#12196) Signed-off-by: -LAN- <laipz8200@outlook.com>
Summary
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Tip
Close issue syntax:
Fixes #<issue number>
orResolves #<issue number>
, see documentation for more details.Screenshots
Checklist
Important
Please review the checklist below before submitting your pull request.
dev/reformat
(backend) andcd web && npx lint-staged
(frontend) to appease the lint gods