-
Notifications
You must be signed in to change notification settings - Fork 344
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
gRPC for RPC #134
Comments
Looks like a good idea. It seems to have a decent language support (not as much as protocol buffers but better than any other rpc implementation out there currently). We could branch off and move to grpc and along with that we can move a lot of our proto messages to services. |
Currently we have gRPC support when Gauge runs in the lsp mode. We should support gRPC for execution and completely remove RPC. For this to be effective, we need to move ensure that Gauge works with both RPC and gRPC for awhile. Some plugins may choose to use gRPC and Gauge should establish a connection with these plugins through gRPC. If the plugin doesn't specify its preference RPC should be used by default. |
The specific issues/tasks which needs to be fixed it order complete this |
The below issues/tasks which has fixed. |
gRPC
(https://github.com/grpc) is becoming popular and it is very useful for RPC, We can use that to avoid lot of boilerplate code that we are using to do RPC over protobuf.The text was updated successfully, but these errors were encountered: