-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
Polish ParallelExectuor constructor into small functions #32191
Conversation
Thanks for your contribution! |
febe376
to
17821c4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
PR types
Others
PR changes
Others
Describe
Polish ParallelExectuor constructor into small functions
1. PR 背景:
ParallelExecutor执行器构造函数逻辑过于复杂,非常不利于后续其他模块复用此代码,比如动转静执行器升级PE工作。
2. PR 内容:
对ParallelExectuor构造函数进行了逻辑拆分,增强代码的可读性和可复用性。本PR为动转静升级PE的前序工作,不涉及新功能增加。
此PR后,PE的构造函数为: