-
Notifications
You must be signed in to change notification settings - Fork 173
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
simple java client #153
simple java client #153
Conversation
Codecov Report
@@ Coverage Diff @@
## main #153 +/- ##
==========================================
+ Coverage 56.47% 56.56% +0.09%
==========================================
Files 51 51
Lines 2215 2215
==========================================
+ Hits 1251 1253 +2
+ Misses 842 841 -1
+ Partials 122 121 -1
Continue to review full report at Codecov.
|
sdk/java-sdk/src/main/java/io/mosn/layotto/v1/LayottoClientImpl.java
Outdated
Show resolved
Hide resolved
sdk/java-sdk/src/main/java/io/mosn/layotto/v1/LayottoClient.java
Outdated
Show resolved
Hide resolved
sdk/java-sdk/src/main/java/io/mosn/layotto/v1/LayottoClientImpl.java
Outdated
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.
LGTM
What this PR does:
add a simple java client , update protobuf and grpc java code, test call rpc successfully.
the client only simply encapsulate the grpc methods , at the same time some methods are not implemented
Which issue(s) this PR fixes:
Fixes #148
Special notes for your reviewer:
Does this PR introduce a user-facing change?: