-
Notifications
You must be signed in to change notification settings - Fork 29.1k
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
Add option for running the logger with flushing to disk after each log call #80862
Comments
@joaomoreno @sandy081 Could you please squeeze this in for September? I would like to enable the extension host to always use synchronous flushing to disk after each log call so I can troubleshoot crashes... |
I think we should do this with an env variable or cli argument, since as a setting we might not be able to read it as early as we'd like at startup and we'd only want this for troubleshooting. My preference would be with an env variable, since that propagates effortlessly. Is that OK @alexandrudima? |
@joaomoreno I would hard-code the extension host to always use synchronous flushing. TBH I don't understand why one would ever use logging without synchronously flushing ^^ |
Because this will block every time a log statement is written. I cannot say how much it will impact the perf but why do not this is optional and enable this only when needed? |
Anyways, I have added a constructor option After we decide between env variable or cli argument, we can create SpdLogService accordingly. |
As discussed exposed |
Thank you! ❤️ |
It is currently very difficult to troubleshoot crashing extension host issues because the logs could be missing data that was not synchronously flushed to disk after each log call.
The text was updated successfully, but these errors were encountered: