Azure / azure-functions-host

The host/runtime that powers Azure Functions
https://functions.azure.com
MIT License
1.92k stars 441 forks source link

Fix lifetime scope of the distributed lock coordinator #2326

Open fabiocav opened 6 years ago

fabiocav commented 6 years ago

The fix in commit f6f5fb36915619140a2e32c18aee358549be0b23 addresses some of the issues we were having with multiple instances of the script host generating lease conflices, but the ideal approach would be to change the lifetime scope of the coordinator so we're not creating a new one with every restart and we don't have the pattern where the factory is disposing of the instance it created.

fabiocav commented 6 years ago

/cc @mathewc