numaproj / numaflow-python

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

chore: refactor into server classes #81

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 #81 (046fd5a) into main (26bc55a) will decrease coverage by 0.71%. The diff coverage is 95.49%.

@@            Coverage Diff             @@
##             main      #81      +/-   ##
==========================================
- Coverage   97.51%   96.80%   -0.71%     
==========================================
  Files          12       14       +2     
  Lines         482      626     +144     
  Branches       31       41      +10     
==========================================
+ Hits          470      606     +136     
- Misses          7       12       +5     
- Partials        5        8       +3     
Impacted Files Coverage Δ
pynumaflow/function/_dtypes.py 98.84% <ø> (ø)
pynumaflow/sink/async_sink.py 93.87% <93.87%> (ø)
pynumaflow/function/async_server.py 94.49% <94.49%> (ø)
pynumaflow/function/multiproc_server.py 97.10% <97.10%> (ø)
pynumaflow/function/__init__.py 100.00% <100.00%> (ø)
pynumaflow/function/server.py 96.77% <100.00%> (-0.79%) :arrow_down:
pynumaflow/sink/__init__.py 100.00% <100.00%> (ø)
pynumaflow/sink/server.py 95.65% <100.00%> (ø)
pynumaflow/types.py 100.00% <100.00%> (ø)

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