nodece commented on PR #22017: URL: https://github.com/apache/pulsar/pull/22017#issuecomment-1927306779
> There's a way to improve the experience for everyone by documenting this. Sure, we require a Pulsar log document, which is an important thing for debugging the Pulsar. > You didn't seem to read my arguments why I'm not supporting this change. Please go back to read https://github.com/apache/pulsar/pull/22017#pullrequestreview-1861988153 carefully. I'm happy to explain in more detail. I know your arguments. Because of consistency, there are also pulsar.log.appender, pulsar.log.dir, pulsar.log.immediateFlush, pulsar.routing.appender.default. Therefore, you suggest that I replace the default `log4j2.yaml`(replace "pulsar.log." -> "my.log."). I know this is a simple thing, but I don't want to do that, and I also want to keep consistency with Apache Pulsar. That's why I made this PR. In my scenario, when the Pulsar is running, I only want to dynamically change the log level for debugging the Pulsar bug. Therefore, I think by modifying the `pulsar.log.level` and `pulsar.log.root.level` values from the info level to the debug level, the Pulsar will print the debug log. Same issue: https://github.com/apache/pulsar/issues/14298 If you have any better ideas, please let me know. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
