The Wayback Machine - http://web.archive.org/web/20201106162328/https://github.com/alibaba/arthas/pull/920
Skip to content
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

improve LoggerCommand #920

Open
wants to merge 1 commit into
base: master
from
Open

improve LoggerCommand #920

wants to merge 1 commit into from

Conversation

@qxo
Copy link
Contributor

@qxo qxo commented Nov 9, 2019

  1. 重构level方法,返回第一个最合适的classloader对应LoggerTypes,默认情况基本不用-c参数,提高日志设置指令可重用度,提升可用性
    根据应用环境的实际情况再调用具体实现类库更新日志级别,而不是之前的不管是否有log4j/logback/log4j分别各调用一次(不存在类库会在日志文件生成异常日志)
  2. 增加-p参数,可根据类名来过滤classloader或loggerName
  3. 日志级别设置时结果多加了各日志类库的对应的日志级别修改结果
    同时输出对应classloader的hashCode,用于recheck: 用来在需要时确认设置的日志级别对应classloader和期望的是否一致
@qxo qxo force-pushed the qxo:improve4logger branch from af50c30 to d7e22d9 Nov 15, 2019
@qxo qxo force-pushed the qxo:improve4logger branch 2 times, most recently from 6dc1bbd to eb9bb53 Dec 8, 2019
@qxo qxo force-pushed the qxo:improve4logger branch 2 times, most recently from b5e915d to 0c4fbf8 Apr 4, 2020
@CLAassistant
Copy link

@CLAassistant CLAassistant commented Jun 12, 2020

CLA assistant check
All committers have signed the CLA.

1. 重构level方法,返回第一个最合适的classloader对应LoggerTypes,默认情况基本不用-c参数,提高日志设置指令可重用度,提升可用性
   根据应用环境的实际情况再调用具体实现类库更新日志级别,而不是之前的不管是否有log4j/logback/log4j分别各调用一次(不存在类库会在日志文件生成异常日志)
2. 增加-p参数,可根据类名来过滤classloader或loggerName
3. 日志级别设置时结果多加了各日志类库的对应的日志级别修改结果
   同时输出对应classloader的hashCode,用于recheck: 用来在需要时确认设置的日志级别对应classloader和期望的是否一致
@qxo qxo force-pushed the qxo:improve4logger branch from 0c4fbf8 to 28c2f0e Jul 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.