Skip to content

Commit

Permalink
Unify and move receive timestamp config
Browse files Browse the repository at this point in the history
Summary: We shouldn't have config in the codec types. Instead solve the plumbing problem more explicitly, and only define the config in one place.

Reviewed By: jbeshay

Differential Revision: D45881730

fbshipit-source-id: fab6c967a38172f16e57a8978b10460fd196902e
  • Loading branch information
Matt Joras authored and facebook-github-bot committed May 17, 2023
1 parent 1386c25 commit 64c6aca
Showing 1 changed file with 2 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -276,9 +276,8 @@ void initializeTransportSettings(HQToolParams& hqUberParams) {

if (FLAGS_use_ack_receive_timestamps) {
hqParams.transportSettings.maybeAckReceiveTimestampsConfigSentToPeer.assign(
{.max_receive_timestamps_per_ack =
FLAGS_max_ack_receive_timestamps_to_send,
.receive_timestamps_exponent = kDefaultReceiveTimestampsExponent});
{.maxReceiveTimestampsPerAck = FLAGS_max_ack_receive_timestamps_to_send,
.receiveTimestampsExponent = kDefaultReceiveTimestampsExponent});
}
} // initializeTransportSettings

Expand Down

0 comments on commit 64c6aca

Please sign in to comment.