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.

Sign up for GitHub

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

Jump to bottom

Apollo Java客户端初始化过程日志打印问题 #3640

Closed
laywin opened this issue Apr 19, 2021 · 3 comments
Closed

Apollo Java客户端初始化过程日志打印问题 #3640

laywin opened this issue Apr 19, 2021 · 3 comments
Labels
area/client apollo-client discussion Categorizes issue as related to discussion

Comments

@laywin
Copy link

laywin commented Apr 19, 2021

Describe the bug
当apollo.bootstrap.eagerLoad.enabled=true时，Apollo初始化过程中的日志均无法输出

To Reproduce
Steps to reproduce the behavior:
当apollo.bootstrap.eagerLoad.enabled=true时，spring boot日志系统并未初始化好，这个过程中产生的日志均无法输出

Expected behavior
建议可以根据类似spring boot中DeferredLog的处理方式来进行这个阶段的日志打印
@nobodyiam nobodyiam added discussion Categorizes issue as related to discussion area/client apollo-client labels Apr 20, 2021
@nobodyiam
Copy link
Member

目前确实存在这个问题，粗看了下 DeferredLog 的实现，大致是先把日志存在内存中，然后在接收到 ApplicationPreparedEvent 后再把日志输出，应该是可以借鉴实现一下的

@klboke klboke mentioned this issue May 10, 2021
Closed
@klboke
Copy link
Contributor

klboke commented May 11, 2021 •
edited
Loading

参考 DeferredLog 的方式，缓存了下日志系统未初始化前的日志，详情见 pr：#3677

@nobodyiam
Copy link
Member

implemented by #3677

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/client apollo-client discussion Categorizes issue as related to discussion
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@nobodyiam @klboke @laywin