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

Introduce worker services and wire them up with meta's REST endpoint #419

Closed
wants to merge 3 commits into from
Closed

Introduce worker services and wire them up with meta's REST endpoint #419

wants to merge 3 commits into from

Conversation

tillrohrmann
Copy link
Contributor

This PR is based on #418. Only the last commit is relevant for this PR.

This commit introduces the worker services which receive WorkerCommands.
The worker commands will be forwarded to the correct partition processor
by proposing the command to the consensus module.

The meta's REST endpoint can communicate with the worker services via an
in-memory channel.

This fixes #408.

The kill command fails an invocation with the error code 10 which
represents the aborted grpc status code. Moreover, it sends an abort
message to the invoker to stop executing this invocation.

If the specified service invocation is not executing (invoked or
suspended), then the command will be ignored.

This fixes #406.
This commit introduces the worker services which receive WorkerCommands.
The worker commands will be forwarded to the correct partition processor
by proposing the command to the consensus module.

The meta's REST endpoint can communicate with the worker services via an
in-memory channel.

This fixes #408.
@slinkydeveloper
Copy link
Contributor

I'm gonna base my code on this PR and then merge all together

@slinkydeveloper
Copy link
Contributor

Merged in #422

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.

Wire up node services with partition processors to be able to propose state machine commands
2 participants