Skip to content
This repository has been archived by the owner on Dec 19, 2018. It is now read-only.

Output to console that the application is shutting down gracefully #794

Closed
mrahhal opened this issue Jun 9, 2016 · 10 comments
Closed

Output to console that the application is shutting down gracefully #794

mrahhal opened this issue Jun 9, 2016 · 10 comments

Comments

@mrahhal
Copy link
Contributor

mrahhal commented Jun 9, 2016

Until recently whenever I wanted to stop my app I felt that ctrl+c felt really slow without a reason, I didn't know that it was canceling the shutdown to wait for the main thread to exit gracefully.

So perhaps output a message after this line of code to say that "Application is shutting down." so that it doesn't feel like nothing is going on?

@davidfowl
Copy link
Member

How long did it take?

@mrahhal
Copy link
Contributor Author

mrahhal commented Jun 9, 2016

Takes a while sometimes, most of the times not so much. The problem is that people won't know what's going on and will just force it to exit (that's what I've been doing). This might or might not be problematic, I use IApplicationLifetime to clean stuff up for example so this got me to think.

@davidfowl
Copy link
Member

This might or might not be problematic, I use IApplicationLifetime to clean stuff up for example so this got me to think.

That makes sense. It really won't help in the general case (since this only helps when looking at the console directly) but I don't see a problem doing this.

@mrahhal
Copy link
Contributor Author

mrahhal commented Jun 9, 2016

Yeah, that's my reasoning also. I'll do a PR.

@davidfowl
Copy link
Member

We'll likely not take this for 1.0.0 since we're pretty much locking down. Also consider this bug that we do plan to fix #770 (which is about sprinkling Console messages that need to be turned off sometimes).

@mrahhal
Copy link
Contributor Author

mrahhal commented Jun 9, 2016

Oh makes sense, then feel free to close this if you want.

@mrahhal
Copy link
Contributor Author

mrahhal commented Jun 9, 2016

@davidfowl I did a PR anyway. Feel free to do whatever is appropriate.

@davidfowl
Copy link
Member

Thanks!

@muratg
Copy link

muratg commented Jun 9, 2016

@Tratcher Could you take a look? Seems simple enough to take in 1.0.0 for me.

@Tratcher
Copy link
Member

Tratcher commented Jun 9, 2016

It looks fine, but it can wait, it's only a cosmetic issue.

@muratg muratg added this to the 1.0.1 milestone Jun 9, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants