chore: refactoring sync and async servers

ashwinidulams commented 1 year ago

changes related to refactoring of sync for map, mapt async for map, mapt and reduce.

codecov[bot] commented 1 year ago

Codecov Report

Merging #80 (c3a3ff4) into main (786b48e) will decrease coverage by 21.07%. The diff coverage is 84.67%.

:exclamation: Current head c3a3ff4 differs from pull request most recent head 5ac2961. Consider uploading reports for the commit 5ac2961 to get more accurate results

@@             Coverage Diff             @@
##             main      #80       +/-   ##
===========================================
- Coverage   97.56%   76.50%   -21.07%     
===========================================
  Files          12       14        +2     
  Lines         493      715      +222     
  Branches       33       56       +23     
===========================================
+ Hits          481      547       +66     
- Misses          7      160      +153     
- Partials        5        8        +3

Impacted Files	Coverage Δ
pynumaflow/function/server.py	`0.00% <ø> (-97.57%)`	:arrow_down:
pynumaflow/function/multiproc_server.py	`61.97% <61.97%> (ø)`
pynumaflow/function/async_server.py	`91.81% <91.81%> (ø)`
pynumaflow/function/sync_server.py	`96.87% <96.87%> (ø)`
pynumaflow/function/__init__.py	`100.00% <100.00%> (ø)`

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

vigith commented 1 year ago

close in favor of #81

numaproj / numaflow-python

chore: refactoring sync and async servers #80

Codecov Report