[
https://issues.apache.org/jira/browse/FELIX-6463?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17424967#comment-17424967
]
Tom Watson commented on FELIX-6463:
-----------------------------------
I'm in favor of reducing the noise of DEBUG because it can become pretty loud
when trying to debug something with much of it being of little value to debug.
I suggest we start gathering log points we think should be moved to TRACE. I'm
less sure on moving stuff into INFO by default if they are really intended to
be debug messages.
> Changing log levels for DS component getState and registration
> --------------------------------------------------------------
>
> Key: FELIX-6463
> URL: https://issues.apache.org/jira/browse/FELIX-6463
> Project: Felix
> Issue Type: Improvement
> Components: Declarative Services (SCR)
> Reporter: Heiko Studt
> Priority: Major
>
> As a developer (having the debugger hat), I need to find the most standard
> problems most easily. Currently, DS services are logging hundreds and
> thousands of lines on startup into DEBUG but nearly nothing into INFO.
> So, in my opinion, Felix should reconsider the log levels of all its DS
> bundle startup loglines. The two most important ones (in my opinion) are
> argued below:
> Beginning with AbstractComponentManager::getState. The "Querying state" log
> should be in level TRACE, as that very state is written six times(!) for
> every state change 3xActivateInteral: (2x inside getState and then again in
> the "Activating" line itself) and analogous 3x for the real state change.
> Here, all four of these "Querying state" lines are superfluous and only
> necessary, if I search for deep felix implementation bugs.
> Reference:
> [https://github.com/apache/felix-dev/blob/3e5671ae7e5107f4f849ef9d5f0a89b1ba9d7439/scr/src/main/java/org/apache/felix/scr/impl/manager/AbstractComponentManager.java#L1427]
> Additionally, the registation changes of a component should be logged into
> INFO, instead of DEBUG. Then, I do not need to activate DEBUG for my DS based
> bundles in the first place (at least for most of my debugging efforts).
> [https://github.com/apache/felix-dev/blob/3e5671ae7e5107f4f849ef9d5f0a89b1ba9d7439/scr/src/main/java/org/apache/felix/scr/impl/manager/RegistrationManager.java#L119]
>
> The two minor changes will improve the debugging experience alot and will
> reduce its needed time significantly. I will be glad to create a pull request
> if this issue is approved.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
