Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Enhancement] StartupTask as async #31

Closed
diegostamigni opened this issue Jul 11, 2019 · 3 comments
Closed

[Enhancement] StartupTask as async #31

diegostamigni opened this issue Jul 11, 2019 · 3 comments

Comments

@diegostamigni
Copy link

Summary

Any reason why there's no StartAsync in IStartupTask? Or maybe an additional IStartupAsyncTask ?

API Changes

IStartupTask.StartAsync or additional IStartupAsyncTask with StartAsync

Intended Use Case

Some startup tasks can be async.

@aritchie
Copy link
Member

I've been debating this. Right now, I'm pretty against doing this because you only get a few seconds in certain background scenarios. Startups needs to be almost instant (no async). The jobs themselves can take a bit longer though.

@aritchie
Copy link
Member

Ya - it is just too dangerous to open this up. Startup tasks were intended to wire up to external events and such. They aren't intended for infrastructure initialization because they need to happen without any perf hit.

@lock
Copy link

lock bot commented Dec 10, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Dec 10, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants