-
Notifications
You must be signed in to change notification settings - Fork 521
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
fix(pd/store): log files do not scroll with the process #2589
Conversation
GRPC Java defaults to using java. til.logging (JUL) as the logging framework. In order to print logs using log4j2.xml configuration rules, the dependency Bridge from JUL to Log4j is used
hugegraph-pd/hg-pd-dist/src/assembly/static/bin/start-hugegraph-pd.sh
Outdated
Show resolved
Hide resolved
@@ -129,7 +129,7 @@ | |||
<!-- Use mixed async way to output logs --> |
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 comments here are somewhat outdated. I found that AsyncLogger is still being used in hugegraph-pd/hg-pd-service/src/main/resources/log4j2.xml. Any context @imbajin?
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 comments here are somewhat outdated. I found that AsyncLogger is still being used in hugegraph-pd/hg-pd-service/src/main/resources/log4j2.xml. Any context @imbajin?
maybe just due to (they) copy the config file from server?
@haohao0103 Running the
|
exec ${JAVA} -Dname="HugeGraphPD" ${JVM_OPTIONS} ${JAVA_OPTIONS} -jar According to this startup command, there needs to be hg-pd-service - *. jar in the lib directory. I guess the target dependency was not generated? |
I discovered an unexpected line break... |
merge it after #2599 |
close #2581
GRPC Java defaults to using java. til.rogging (JUL) as its logging framework,