Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Optimize Client with Less Socket Connections #70

Merged
merged 2 commits into from
Dec 28, 2023
Merged

Optimize Client with Less Socket Connections #70

merged 2 commits into from
Dec 28, 2023

Conversation

jasondu7297
Copy link
Collaborator

This PR accomplishes:

  • Decreases the number of times we open socket connection

@jasondu7297 jasondu7297 self-assigned this Dec 18, 2023
adriandavila
adriandavila approved these changes Dec 18, 2023
View reviewed changes
Copy link
Collaborator

@adriandavila adriandavila left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice nice. Should improve our client throughput quite substantially. From your initial experimentation, looks like it overwhelmed the host so we'll hold off on merging this change for now.

@jasondu7297 jasondu7297 merged commit a71be05 into ortoa-tee Dec 28, 2023
@jasondu7297 jasondu7297 deleted the jasondu/client-optimization branch December 28, 2023 16:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@adriandavila adriandavila adriandavila approved these changes

Assignees

@jasondu7297 jasondu7297

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jasondu7297 @adriandavila