Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

RestServer 相关是有一定原因和考虑意义的, 另外它可以全局调整 rest-api 的所有日志级别.

这个改不改可以看看其他同学意见

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这块我能理解，但是看已有代码也不够统一

嗯嗯后面有些修正是需要的 ✓ (那种一般是更新类名的时候没注意一起调整)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

RestServer 相关是有一定原因和考虑意义的, 另外它可以全局调整 rest-api 的所有日志级别.

这个改不改可以看看其他同学意见

好像可以根据包路径配置日志的

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

RestServer 相关是有一定原因和考虑意义的, 另外它可以全局调整 rest-api 的所有日志级别.
这个改不改可以看看其他同学意见

好像可以根据包路径配置日志的

是可以, 当然我指的是运行时动态调整日志级别 (不过那个也可以单独修改)

这里复用一个类名是不是有什么其他历史考量呢, 需要确认一下 @javeme