Skip to content

Commit 91b70da

Browse files
authored
Merge pull request #12354 from jacquesqiao/parallel-executor-support-prefetch
distribute lookup table work with parallel executor
2 parents d7e08c5 + 64e7902 commit 91b70da

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

paddle/fluid/framework/details/multi_devices_graph_builder.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -259,7 +259,7 @@ std::unique_ptr<ir::Graph> MultiDevSSAGraphBuilder::Apply(
259259
result.Set("ops", new GraphOps);
260260

261261
// find send/recv vars so that we can place the distributed training
262-
// realted op in the place 0
262+
// related op in the place 0
263263
auto send_vars = FindDistTrainSendVars(sorted_ops);
264264
auto recv_vars = FindDistTrainRecvVars(sorted_ops);
265265

python/paddle/fluid/transpiler/distribute_transpiler.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -779,7 +779,9 @@ def _replace_lookup_table_op_with_prefetch(self, program,
779779
outputs={"Out": prefetch_output_vars},
780780
attrs={
781781
"epmap": pserver_endpoints,
782-
RPC_OP_ROLE_ATTR_NAME: RPC_OP_ROLE_ATTR_VALUE
782+
# FIXME(qiao) temporarily disable this config because prefetch
783+
# is not act as other rpc op, it's more like a forward op
784+
# RPC_OP_ROLE_ATTR_NAME: RPC_OP_ROLE_ATTR_VALUE
783785
})
784786

785787
# insert concat_op

0 commit comments

Comments
 (0)