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

[ISSUE #2466] Add debug log #2468

Merged
merged 2 commits into from
Mar 18, 2022
Merged

Conversation

li-xiao-shuang
Copy link
Contributor

fix : #2466

Describe what this PR does / why we need it

Does this pull request fix one issue?

Describe how you did it

Describe how to verify it

Special notes for reviews

@steverao
Copy link
Collaborator

Would it be better to add dataId and group as well?

@li-xiao-shuang
Copy link
Contributor Author

Would it be better to add dataId and group as well?

ok

@steverao
Copy link
Collaborator

steverao commented Mar 18, 2022

Are we adding debug logs or some info logs to help users see the configuration loading output in real time? Now use spring.config.import without any log output, you can try

@li-xiao-shuang
Copy link
Contributor Author

li-xiao-shuang commented Mar 18, 2022

Are we adding debug logs or some info logs to help users see the configuration loading output in real time? Now use spring.config.import without any log output, you can try

I don't know much about Spring-cloud-alibaba, can you explain in detail, what is use spring.config.import?

@steverao
Copy link
Collaborator

This can be modified later!

@steverao steverao merged commit 1ab8b13 into alibaba:2021.x Mar 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add NacosConfigDataLoader log
2 participants