Revise logging usage in openquake
Bug #833900 reported by
Anton Gritsay
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
OpenQuake (deprecated) |
Won't Fix
|
High
|
Unassigned |
Bug Description
1. Write a wiki page with a clear explanation of logging level and names meanings and guidelines of using them.
2. Make logging in openquake project follow that guidelines.
Changed in openquake: | |
milestone: | 0.4.3 → 0.4.4 |
Changed in openquake: | |
milestone: | 0.4.4 → 0.4.5 |
Changed in openquake: | |
assignee: | Benjamin Max Wyss III (bmwyss) → nobody |
milestone: | 0.4.5 → 0.4.7 |
Changed in openquake: | |
milestone: | 0.5.0 → 0.6.0 |
Changed in openquake: | |
milestone: | 0.6.0 → 0.6.1 |
Changed in openquake: | |
status: | Confirmed → Won't Fix |
To post a comment you must log in.
Regarding info/debug/warn levels and logging performance
16:05 < angri> beatpanic: tested with deterministic_ event_based_ risk: takes 2
branch
example produces crazy flood of log messages from java and from
serializer so it is almost two times slower with debug enabled
neglectable
'info' logging level instead of debug? because IIRC there
are messages useful for the user like /bugs.launchpad .net/openquake/ +bug/838984 but they
could be logged as 'info' I guess
min 56 sec on master on my machine and 3 min 2 sec on logging
16:05 < angri> even with --debug=debug
16:07 < angri> but it may be much slower for some other tests. simplecase for
16:09 < angri> but when there are not tens thousands of log messages or they
are suppressed by --debug=warn the performance difference is
16:11 < beatpanic> angri, okay, should we maybe change some meaningful messages in
https:/
our users are used to run --debug=debug on command line to have informational messages, given the "discussion" on the channel we could convert the most important 'debug' messages to less 'loud log level', like 'info' to avoid logging messages pollution and performance problems