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

[Dataflow Streaming Engine] Turn on --useWindmillIsolatedChannels and --useSeparateWindmillHeartbeatStreams by default #31899

Closed
wants to merge 1 commit into from

Conversation

arunpandianp
Copy link
Contributor

Turn on --useWindmillIsolatedChannels and --useSeparateWindmillHeartbeatStreams by default on streaming engine

useWindmillIsolatedChannels: Each windmill grpc stream will use a separate
underlying HTTP2 connection

useSeparateWindmillHeartbeatStreams: Separate windmill GetData streams
will be used for heartbeats and key data requests

If required the features can be turned off by passing --useWindmillIsolatedChannels=false and --useSeparateWindmillHeartbeatStreams=false

…rtbeatStreams` by default on streaming engine

useWindmillIsolatedChannels: Each windmill grpc stream will use a separate
underlying HTTP2 connection

useSeparateWindmillHeartbeatStreams: Separate windmill GetData streams
will be used for heartbeats and key data requests

If required the features can be turned off using `--useWindmillIsolatedChannels=false` and `--useSeparateWindmillHeartbeatStreams=false`
@arunpandianp
Copy link
Contributor Author

R: @scwhittle

Copy link
Contributor

Stopping reviewer notifications for this pull request: review requested by someone other than the bot, ceding control. If you'd like to restart, comment assign set of reviewers

@scwhittle
Copy link
Contributor

Pending ongoing discussions internally on how to deploy

@scwhittle
Copy link
Contributor

Closing as this is being done with new job setting path.

@scwhittle scwhittle closed this Sep 26, 2024
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