-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
enhance: streaming node client implementation #34467
enhance: streaming node client implementation #34467
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: chyezh The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
24637c1
to
6f8ccf5
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #34467 +/- ##
==========================================
+ Coverage 80.69% 84.38% +3.69%
==========================================
Files 1127 900 -227
Lines 139341 116880 -22461
==========================================
- Hits 112443 98631 -13812
+ Misses 22613 13886 -8727
- Partials 4285 4363 +78
|
6f8ccf5
to
a9347c7
Compare
a9347c7
to
d2e4c38
Compare
d2e4c38
to
7ccdac6
Compare
- add two grpc resolver (by session and by streaming coord assignment service) - add one grpc balancer (by serverID and roundrobin) - add lazy conn to avoid block by first service discovery - add some utility function for streaming service - fix some bug Signed-off-by: chyezh <chyezh@outlook.com>
- add streaming node grpc client wrapper - add unittest for streaming node grpc client side Signed-off-by: chyezh <chyezh@outlook.com>
7ccdac6
to
08f186f
Compare
@chyezh E2e jenkins job failed, comment |
close to avoid generate extra ci. |
issue: #33285