search

numaproj / numaflow-python

Numaflow Python SDK
Apache License 2.0
53 stars 18 forks source link

feat: introduce handshake to client and gRPC server #89

Closed kohlisid closed 1 year ago

kohlisid commented 1 year ago

Kindly explain what this PR does.

codecov[bot] commented 1 year ago

Codecov Report

Merging #89 (3a730fc) into main (6dcd082) will decrease coverage by 0.22%. The diff coverage is 95.16%.

@@            Coverage Diff             @@
##             main      #89      +/-   ##
==========================================
- Coverage   97.42%   97.20%   -0.22%     
==========================================
  Files          14       16       +2     
  Lines         659      716      +57     
  Branches       41       45       +4     
==========================================
+ Hits          642      696      +54     
- Misses          9       10       +1     
- Partials        8       10       +2
Impacted Files Coverage Δ
pynumaflow/info/server.py 90.62% <90.62%> (ø)
pynumaflow/function/async_server.py 97.65% <100.00%> (+0.03%) :arrow_up:
pynumaflow/function/multiproc_server.py 97.36% <100.00%> (-0.04%) :arrow_down:
pynumaflow/function/server.py 97.22% <100.00%> (+0.07%) :arrow_up:
pynumaflow/info/types.py 100.00% <100.00%> (ø)
pynumaflow/sink/async_sink.py 94.11% <100.00%> (+0.24%) :arrow_up:
pynumaflow/sink/server.py 95.83% <100.00%> (+0.18%) :arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more