-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
Document error for agent config file 's config description #3943
Conversation
logging.dir 's Description is an Error .Default value is not System.out. It is a dir.
logging.dir 's Description is an Error .Default value is not System.out. It is a dir.
Update README.md
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the description is an Error .In fact, the log will be output to the "agentJarDir/logs "
So It is needed to be fixed .
@@ -93,7 +93,7 @@ property key | Description | Default | | |||
`logging.level`|The log level. Default is debug.|`DEBUG`| | |||
`logging.file_name`|Log file name.|`skywalking-api.log`| | |||
`logging.output`| Log output. Default is FILE. Use CONSOLE means output to stdout. |`FILE`| | |||
`logging.dir`|Log files directory. Default is blank string, means, use "system.out" to output logs.|`""`| | |||
`logging.dir`|Log files directory. Default is blank string, means, use "{theSkywalkingAgentJarDir}/logs " to output logs. {theSkywalkingAgentJarDir} is the directory where the skywalking agent jar file is located |`""`| |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the description is an Error .In fact, the log will be output to the "agentJarDir/logs "
The class comment is wrong too, org.apache.skywalking.apm.agent.core.conf.Config#Logging#DIR. Could you fix this too? |
the comments is an Error . fix it .
ok . I pull an other pull request |
This one is good, you don't need a new one. |
Bug fix
Documents Error In v6.5 ? #3942
In
https://github.com/apache/skywalking/blob/v6.5.0/docs/en/setup/service-agent/java-agent/README.md