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

[Part 2] Introducing Task Executor Gateway #146

Merged
merged 1 commit into from
Mar 21, 2022
Merged

[Part 2] Introducing Task Executor Gateway #146

merged 1 commit into from
Mar 21, 2022

Conversation

sundargates
Copy link
Collaborator

The goal of the gateway is to provide an interface for the control plane to communicate with the task executor. The control plane is expected to communicate with the task executor using RPC instead of HTTP as the task executor is not exposed outside.

Checklist

  • ./gradlew build compiles code correctly
  • Added new tests where applicable
  • ./gradlew test passes all tests
  • Extended README or added javadocs where applicable
  • Added copyright headers for new files from CONTRIBUTING.md

@sundargates
[Part 2] Introducing Task Executor Gateway
ea31168 
The goal of the gateway is to provide an interface for the control plane to communicate with the task executor. The control plane is expected to communicate with the task executor using RPC instead of http as the task executor is not exposed outside.
@sundargates sundargates merged commit a64e8d1 into Netflix:master Mar 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@calvin681 calvin681 calvin681 approved these changes

@codyrioux codyrioux Awaiting requested review from codyrioux

@nickmahilani nickmahilani Awaiting requested review from nickmahilani

@jeffchao jeffchao Awaiting requested review from jeffchao

@piygoyal piygoyal Awaiting requested review from piygoyal

@Andyz26 Andyz26 Awaiting requested review from Andyz26 Andyz26 is a code owner

@hmitnflx hmitnflx Awaiting requested review from hmitnflx hmitnflx is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sundargates @calvin681