-
Notifications
You must be signed in to change notification settings - Fork 40
Added gRPC service generation to the compiler plugin #428
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please check comments
|
||
public fun <R> callServerStreaming(rpcCall: RpcCall): Flow<R> | ||
} | ||
//@ExperimentalRpcApi |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please drop commented code
value = value, | ||
) | ||
|
||
private fun IrType.unwrapFlow(): IrType { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The class is huge. Please check what can be extracted as utility methods
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I still think that one big file is better DX, for the current size at least.
And while I'm the only person making bug changes here, I'd like to have like this. I have in mind splitting it, but only when it's size becomes an issue during the development
Subsystem
Compiler Plugin, gRPC
Problem Description
We have all platform bindings for the gRPC, but no service generation
Solution
Add service generations and a
MessageCodec
locator:MessageCodecResolver
inGrpcClient
/GrpcServer
WithCodec
annotation (default for protobuf messages)And we now have fully working gRPC services with arbitrary serilization.
For next PRs: