Skip to content

Replace all instances of HeaderClientChannel::newChannel with #418

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

paulcruz74
Copy link
Contributor

Summary:
This is for the purpose of testing a hypothesis that some internal difference
between HeaderClientChannel and RocketClientChannel is causing high
latency/hangs during thrift requests to FBOSS services in OSS.

Differential Revision: D73672539

paulcruz74 added 2 commits May 1, 2025 14:38
…ok#416)

Summary:

This change is in order to test a hypothesis that some difference in client
creation between QsfpClient.cpp and ThriftServiceClient.cpp is responsible for
additional latency when executing thrift requests in an OSS context.

Differential Revision: D73670141
Summary:
This is for the purpose of testing a hypothesis that some internal difference
between HeaderClientChannel and RocketClientChannel is causing high
latency/hangs during thrift requests to FBOSS services in OSS.

Differential Revision: D73672539
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D73672539

paulcruz74 added a commit to paulcruz74/fboss that referenced this pull request May 1, 2025
…ok#418)

Summary:

This is for the purpose of testing a hypothesis that some internal difference
between HeaderClientChannel and RocketClientChannel is causing high
latency/hangs during thrift requests to FBOSS services in OSS.

Differential Revision: D73672539
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants