-
Notifications
You must be signed in to change notification settings - Fork 8.8k
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
HDDS-1720 : Add ability to configure RocksDB logs for Ozone Manager. #1538
Conversation
/label ozone |
💔 -1 overall
This message was automatically generated. |
/retest |
1 similar comment
/retest |
Can you please add a test case that proves that RocksDB actually produces logs that we can see. There is a log listener class in the Hadoop. You can use them, or create a test and then grep for some of the log statements. Otherwise the change looks quite good to me. |
Thanks for the review @anuengineer . I have added unit test for verifying RocksDB logging. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
Thank you for the contribution. I have committed this to the trunk. |
With this patch, RocksDB logging can be configured for OM. If needed, we can write RocksDB logs to a separate file using a separate appender in log4j.properties.