abp icon indicating copy to clipboard operation
abp copied to clipboard

Published 20 hours ago • verified publisher icon abpframework

Enqueue background jobs with `executionTime`.

Open gdlcf88 opened this issue 3 years ago • 4 comments

#10311

gdlcf88 avatar Oct 13 '21 18:10 gdlcf88

@hikalkan What do you think?

maliming avatar Oct 18 '21 06:10 maliming

We can just calculate the delay (as executionTime - Clock.Now) and pass to the current method. Why we should define another method?

hikalkan avatar Nov 05 '21 11:11 hikalkan

executionTime - Clock.Now is fine. However, the final (first) execution time may depend on the job inserted time if we use the delay. I'm not sure how long it will spend. It is not rigorous enough when we have an absolute execution time.

gdlcf88 avatar Nov 05 '21 12:11 gdlcf88

There is also a BackgroundJob Manager implementation for Hangfire (https://github.com/abpframework/abp/pull/10292) that will require to be updated for this specific change. Would also consider updating the documentation with this change

gdeswardt avatar Jan 01 '22 08:01 gdeswardt

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar May 15 '23 15:05 CLAassistant