faabric icon indicating copy to clipboard operation
faabric copied to clipboard

scheduler: tag executors with app id

Open csegarragonz opened this issue 9 months ago • 1 comments

Thus far, we tagged executors with the (user, function) pair. This is needed in multi-threaded environments, where different threads must share the same executor. However, using only the (user, function) pair prevents the execution concurrent applications with the same pair.

Instead, the correct thing to do is to track applications using the (user, function) pair in conjunction with the application id.

csegarragonz avatar May 06 '24 20:05 csegarragonz

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 81.33%. Comparing base (ddba013) to head (c228c0f).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #434   +/-   ##
=======================================
  Coverage   81.33%   81.33%           
=======================================
  Files         117      117           
  Lines        8233     8233           
=======================================
  Hits         6696     6696           
  Misses       1537     1537           

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar May 06 '24 20:05 codecov[bot]