Skip to content

Commit

Permalink
#702: TestTermChaining: ensure that the callback is executed on the s…
Browse files Browse the repository at this point in the history
…ame node as the root for the reduce
  • Loading branch information
nmm0 committed Aug 25, 2020
1 parent fb31ca1 commit 9cc2971
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions tests/unit/termination/test_term_chaining.cc
Original file line number Diff line number Diff line change
Expand Up @@ -165,7 +165,7 @@ struct TestTermChaining : TestParallelHarness {
EpochType epoch2 = theTerm()->makeEpochCollective();
vt::theMsg()->pushEpoch(epoch2);
auto msg2 = makeMessage<ChainReduceMsg>(theContext()->getNode());
auto cb = vt::theCB()->makeFunc<ChainReduceMsg>( &test_handler_reduce );
auto cb = vt::theCB()->makeSend<ChainReduceMsg, test_handler_reduce>( 0 );
chain.add(epoch2, theCollective()->global()->reduce< vt::collective::None >(0, msg2.get(), cb));
vt::theMsg()->popEpoch(epoch2);
vt::theTerm()->finishedEpoch(epoch2);
Expand All @@ -188,7 +188,7 @@ struct TestTermChaining : TestParallelHarness {
EpochType epoch2 = theTerm()->makeEpochRooted();
vt::theMsg()->pushEpoch(epoch2);
auto msg2 = makeMessage<ChainReduceMsg>(theContext()->getNode());
auto cb = vt::theCB()->makeFunc<ChainReduceMsg>( &test_handler_reduce );
auto cb = vt::theCB()->makeSend<ChainReduceMsg, test_handler_reduce>( 0 );
chain.add(epoch2, theCollective()->global()->reduce< vt::collective::None >(0, msg2.get(), cb));
vt::theMsg()->popEpoch(epoch2);
vt::theTerm()->finishedEpoch(epoch2);
Expand Down

0 comments on commit 9cc2971

Please sign in to comment.