Skip to content

[fleet_executor] Implementation of the message bus, the carrier and part of the interceptor #37049

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

Merged
merged 20 commits into from
Nov 10, 2021

Conversation

FeixLiu
Copy link
Contributor

@FeixLiu FeixLiu commented Nov 9, 2021

PR types

Others

PR changes

Others

Describe

Implementation of the message bus, the carrier and part of the interceptor (communication relating part).

UT

Local test with GLOG_v=3
image

Note: this UT based on the ip of the network card, so I didn't push the test file to this PR. But do the test locally (as above). You can also test this on your machine codes. Replace this CMakeLists with the CMakeLists in the Gist. Put the test.cc file in the Gist under paddle/fluid/distributed/fleet_executor/ folder.

@paddle-bot-old
Copy link

paddle-bot-old bot commented Nov 9, 2021

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

@FeixLiu FeixLiu requested a review from wangxicoding November 9, 2021 01:15
@FeixLiu FeixLiu force-pushed the codes_for_message_bus branch from cf15804 to 84ea158 Compare November 9, 2021 03:27
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 072e780 into PaddlePaddle:develop Nov 10, 2021
@FeixLiu FeixLiu deleted the codes_for_message_bus branch November 12, 2021 06:26
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.

3 participants