User tests: Successful: Unsuccessful:
Pull Request for Issue #34908
I have removed messagequeue logger from bootstrap, and added changes from #34912 by @brianteeman
Please follow #34908
Logging of jerror
can be configured in global configuration:
Status | New | ⇒ | Pending |
Category | ⇒ | Libraries |
That's a too big a b/c break at this stage.
I think no one will cry for this kind of logging (with use of an alert),
anyway it is useless for an average User :)
As you decide.
@brianteeman PR also okay, but then we need another category, that not a jerror
and not update
.
Or can just remove the log from that place, in general
I think no one will cry for this kind of logging (with use of an alert),
anyway it is useless for an average User :)
I agree but I know several extension take advantage of it. It's just not worth the pain.
@brianteeman PR also okay, but then we need another category, that not a jerror and not update.
agreed
Or can just remove the log from that place, in general
Probably worth keeping in case of any very weird edge cases.
Status | Pending | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2021-07-27 07:05:13 |
Closed_By | ⇒ | Fedik | |
Labels |
Added:
?
|
I think this change is too much. That's a too big a b/c break at this stage. I think @brianteeman 's PR is the right lines. We probably just need a general log file to be created. There are other system level logs we throw in various places for the category. Obviously I understand jerror is not good.