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.

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

feat(rust): make portal handshake optional #8777

Merged
merged 1 commit into from
Feb 5, 2025

Conversation

SanjoDeundiak
Copy link
Member

@SanjoDeundiak SanjoDeundiak commented Jan 27, 2025

  1. Add --skip-handshake flag to tcp inlets&outlets, that makes handshake non-blocking to optimize latency (but possibly sacrificing throughput)
  2. Add --enable-nagle flag to tcp inlets&outlets. By default we set nodelay flag on TCP streams, but that may sometimes be unwanted (for high throughput use cases)

@SanjoDeundiak SanjoDeundiak force-pushed the sanjo/portal_handshake branch 6 times, most recently from 585ac2c to 2346830 Compare February 3, 2025 15:06
@SanjoDeundiak SanjoDeundiak marked this pull request as ready for review February 3, 2025 15:07
@SanjoDeundiak SanjoDeundiak requested a review from a team as a code owner February 3, 2025 15:07
@SanjoDeundiak SanjoDeundiak force-pushed the sanjo/portal_handshake branch from 2346830 to 102fe51 Compare February 3, 2025 18:42
@SanjoDeundiak SanjoDeundiak force-pushed the sanjo/portal_handshake branch from f051d92 to f9f12e6 Compare February 5, 2025 17:55
@SanjoDeundiak SanjoDeundiak added this pull request to the merge queue Feb 5, 2025
Merged via the queue into develop with commit f9f12e6 Feb 5, 2025
75 checks passed
@SanjoDeundiak SanjoDeundiak deleted the sanjo/portal_handshake branch February 5, 2025 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants