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

Overrding all classloaders HA settings by -D params #437

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

zhaoxiaojie0415
Copy link

As comment in #366 (comment) #

I think when we want to override properties by -D , it need to take effect in all classloaders?

#366

@skybber
Copy link
Contributor

skybber commented Sep 22, 2022

Dou you know, that you can put configuration file to project resources?

@zhaoxiaojie0415
Copy link
Author

Dou you know, that you can put configuration file to project resources?

Yes I did. In my case, I use the propertiesFilePath command line args to find the hotswap-agent.properties, which is prior to the file in project resources.

I just notice the pull request 366 , and find it can not adjust the logger level entirely.

Well, I am not sure this pr is significant. --!

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.

2 participants