-
Notifications
You must be signed in to change notification settings - Fork 93
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
Task messages should be recorded in suite DBs. #1490
Comments
@hjoliver I am currently in the middle of adding some new tables and doing some refactor to |
@matthewrmshin - are you working on #864? I'll attempt to put a bit of thought into this tomorrow... |
E.g. |
Non-output task messages can be sent with NORMAL, WARNING, or CRITICAL priority. This might seem a more-or-less useless feature. Thinking back, the intention was to log all messages in case of typos that caused output messages not to be recognized as outputs, and to allow deliberate recording of task errors centrally back when task job logs were not so easily accessed. There is probably little need for this now (although the first reason is still valid) but given that we do support such messages they should really be recorded in the DB as well as logged. Easy to do - the "message" column in the Changing the Issue title to "all messages" as there's no reason to single out WARNINGs... |
(currently they're not).
The text was updated successfully, but these errors were encountered: