Azure / azure-functions-python-worker

Python worker for Azure Functions.
http://aka.ms/azurefunctions
MIT License
335 stars 103 forks source link

Handling multi worker scenerio for pystein #1142

Closed gavin-aguiar closed 1 year ago

gavin-aguiar commented 1 year ago

Description

Change to index the function in the load request. This is for cases where we have more than one worker. Since only the first worker calls the metadata request and the subsequent workers call the load request directly, we need to index the functions for worker2 and onwards in the load request.

Fixes #


PR information

Quality of Code and Contribution Guidelines

codecov[bot] commented 1 year ago

Codecov Report

Merging #1142 (02499f3) into dev (28e5fdb) will decrease coverage by 0.10%. The diff coverage is 84.61%.

:exclamation: Current head 02499f3 differs from pull request most recent head 01b6ee0. Consider uploading reports for the commit 01b6ee0 to get more accurate results

@@            Coverage Diff             @@
##              dev    #1142      +/-   ##
==========================================
- Coverage   84.97%   84.86%   -0.11%     
==========================================
  Files          32       32              
  Lines        1817     1824       +7     
  Branches      275      276       +1     
==========================================
+ Hits         1544     1548       +4     
- Misses        203      204       +1     
- Partials       70       72       +2     
Flag Coverage Δ
unittests 84.81% <84.61%> (-0.11%) :arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
azure_functions_worker/loader.py 84.70% <ø> (-0.36%) :arrow_down:
azure_functions_worker/dispatcher.py 88.86% <81.81%> (-0.36%) :arrow_down:
azure_functions_worker/functions.py 96.27% <100.00%> (+0.06%) :arrow_up:
azure_functions_worker/extension.py 98.16% <0.00%> (-0.92%) :arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.