-
Notifications
You must be signed in to change notification settings - Fork 34
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
Add seed sample #598
Add seed sample #598
Conversation
...er/modules/process/src/main/scala/example/seed/server/process/AvroPeopleServiceHandler.scala
Outdated
Show resolved
Hide resolved
…ample/seed/server/process/AvroPeopleServiceHandler.scala Co-Authored-By: AdrianRaFo <15971742+AdrianRaFo@users.noreply.github.com>
Codecov Report
@@ Coverage Diff @@
## master #598 +/- ##
=======================================
Coverage 82.36% 82.36%
=======================================
Files 62 62
Lines 964 964
Branches 12 12
=======================================
Hits 794 794
Misses 170 170 Continue to review full report at Codecov.
|
modules/examples/seed/server/modules/protocol_avro/src/main/scala/People.scala
Show resolved
Hide resolved
modules/examples/seed/server/modules/protocol_avro/src/main/scala/PeopleService.scala
Show resolved
Hide resolved
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.
Left some comments but it looks overall. Thanks
...ent/modules/process/src/main/scala/example/seed/client/process/AvroPeopleServiceClient.scala
Outdated
Show resolved
Hide resolved
...nt/modules/process/src/main/scala/example/seed/client/process/ProtoPeopleServiceClient.scala
Show resolved
Hide resolved
...nt/modules/process/src/main/scala/example/seed/client/process/ProtoPeopleServiceClient.scala
Show resolved
Hide resolved
|
||
abstract class ServerBoot[F[_]: ConcurrentEffect] { | ||
|
||
def runProgram(args: List[String]): F[ExitCode] = |
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.
Same as the client program, you can make ServerBoot
extends IOApp
and rename this to def run(args: List[String]): IO[ExitCode]
...r/modules/process/src/main/scala/example/seed/server/process/ProtoPeopleServiceHandler.scala
Outdated
Show resolved
Hide resolved
Co-Authored-By: AdrianRaFo <15971742+AdrianRaFo@users.noreply.github.com>
Co-Authored-By: AdrianRaFo <15971742+AdrianRaFo@users.noreply.github.com>
Co-Authored-By: AdrianRaFo <15971742+AdrianRaFo@users.noreply.github.com>
Co-Authored-By: AdrianRaFo <15971742+AdrianRaFo@users.noreply.github.com>
Co-Authored-By: AdrianRaFo <15971742+AdrianRaFo@users.noreply.github.com>
Co-Authored-By: AdrianRaFo <15971742+AdrianRaFo@users.noreply.github.com>
Co-Authored-By: AdrianRaFo <15971742+AdrianRaFo@users.noreply.github.com>
Co-Authored-By: AdrianRaFo <15971742+AdrianRaFo@users.noreply.github.com>
Co-Authored-By: AdrianRaFo <15971742+AdrianRaFo@users.noreply.github.com>
@fedefernandez @juanpedromoreno ready for a second review |
Add a new sample to Mu based on:
https://github.com/AdrianRaFo/RPC-Avro-Seed
https://github.com/AdrianRaFo/RPC-Proto-Seed
with the minimal client and server on both
avro
andprotobuf