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

[Fleet Executor] Export task node #37509

Merged
merged 1 commit into from
Nov 25, 2021

Conversation

LiYuRio
Copy link
Contributor

@LiYuRio LiYuRio commented Nov 24, 2021

PR types

New features

PR changes

Others

Describe

新增TaskNodeDesc,用于自动并行上游向执行器传递任务信息。

@paddle-bot-old
Copy link

Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.


namespace paddle {
namespace framework {
class OperatorBase;
}
namespace distributed {

class TaskNodeDesc final {
Copy link
Contributor

Choose a reason for hiding this comment

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

这个和TaskNode有啥区别,为啥感觉可以直接用TaskNode😳

Copy link
Contributor Author

Choose a reason for hiding this comment

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

没必要没必要,想多了,合一起了。

@LiYuRio LiYuRio force-pushed the dev_task_node branch 3 times, most recently from 519647d to 3d11462 Compare November 24, 2021 09:47
@LiYuRio LiYuRio changed the title [Fleet Executor] Create task node desc [Fleet Executor] Export task node Nov 24, 2021
Copy link
Contributor

@gongweibao gongweibao left a comment

Choose a reason for hiding this comment

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

LGTM

@gongweibao gongweibao merged commit 3f815e7 into PaddlePaddle:develop Nov 25, 2021
Zjq9409 pushed a commit to Zjq9409/Paddle that referenced this pull request Dec 10, 2021
@LiYuRio LiYuRio deleted the dev_task_node branch December 31, 2021 10:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants