-
Notifications
You must be signed in to change notification settings - Fork 59
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
Extend pie to be able to work with gRPC #10
Comments
This would be really awesome to have, particularly for stream-based RPC. |
any progress on this ? I was close to create an issue with this topic ;) |
No progress, at the time, grpc was still a bit of a moving target. It's nicely released now, so I should look at this again. |
I've just implemented a gRPC transport for a plugin which I previously used pie (it had some error, which occasionally blocked the connection, which blocked everything). |
@tgulacsi and @natefinch can you please help with an answer: Why would you choose communicating over stdin/stdout instead of with Unix domain sockets? Is either method more reliable? |
No, I think they're the same regarding reliability, just that stdin/stdout
is already there, needs no extra setup (unix domain socket needs to create
a file somewhere, or an abstract socket (Linux specific and less secure))
and more platform-agnostic.
Dmitriy <[email protected]> ezt írta (időpont: 2018. aug. 2., Cs,
4:07):
… @tgulacsi <https://github.com/tgulacsi> and @natefinch
<https://github.com/natefinch> can you please help with an answer: Why
would you choose communicating over stdin/stdout instead of with Unix
domain sockets? Is either method more reliable?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#10 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AAPoSj6-FBgVayI9pFNeGdGysIjpUm3kks5uMl7PgaJpZM4G6ome>
.
|
Yeah, mostly because the performance of Unix domain sockets on windows is really bad. ;) |
gRPC is really nice and really fast and has a ton of cross platform support (obviously way more than go-compatible RPC). Enabling gRPC would make a lot of interop problems with pie go away.
The text was updated successfully, but these errors were encountered: