Skip to content

Commit

Permalink
Fix traffic statistics in TwoPartyTupleGenerator (#160)
Browse files Browse the repository at this point in the history
Summary:
Pull Request resolved: #160

gorel raised the issue in D34779616 that the number of bytes sent/received was roughly the same for publisher and partner, whereas they should just be inverted.

This should fix it.

Reviewed By: chualynn

Differential Revision: D35199261

fbshipit-source-id: 9e9a4a22122812ce107a20b9b45e575a789c0dd7
  • Loading branch information
Elliott Lawrence authored and facebook-github-bot committed Mar 28, 2022
1 parent 8c198fd commit 0121f5f
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion fbpcf/engine/tuple_generator/TwoPartyTupleGenerator.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -129,7 +129,7 @@ std::pair<uint64_t, uint64_t> TwoPartyTupleGenerator::getTrafficStatistics()
auto senderStats = senderRcot_->getTrafficStatistics();
auto receiverStats = receiverRcot_->getTrafficStatistics();
rst.first += senderStats.first + receiverStats.first;
rst.second += senderStats.first + receiverStats.second;
rst.second += senderStats.second + receiverStats.second;

return rst;
}
Expand Down

0 comments on commit 0121f5f

Please sign in to comment.