-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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 logging to capture jobControlId #16780
Fix logging to capture jobControlId #16780
Conversation
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.
LGTM, thanks for the contribution!
Please fix the style, you can get related information here https://docs.alluxio.io/os/user/edge/en/contributor/Code-Conventions.html |
alluxio-bot, merge this please |
alluxio-bot, cherry-pick this to branch-2.8 please |
Auto cherry-pick unsuccessful: stdout: <> |
Alluxio/Community#643 Fix logging to capture jobControlId this will help in getting jobControlId in the logs, and will help in getting status information about the job its just log level changes pr-link: Alluxio#16780 change-id: cid-551cf01210bb24d39239199adeb398201f67c184
### What changes are proposed in this pull request? Alluxio/Community#643 Fix logging to capture jobControlId ### Why are the changes needed? this will help in getting jobControlId in the logs, and will help in getting status information about the job ### Does this PR introduce any user facing changes? its just log level changes pr-link: Alluxio#16780 change-id: cid-551cf01210bb24d39239199adeb398201f67c184
### What changes are proposed in this pull request? Alluxio/Community#643 Fix logging to capture jobControlId ### Why are the changes needed? this will help in getting jobControlId in the logs, and will help in getting status information about the job ### Does this PR introduce any user facing changes? its just log level changes pr-link: Alluxio#16780 change-id: cid-551cf01210bb24d39239199adeb398201f67c184
Cherry-pick of existing commit. orig-pr: #16780 orig-commit: 412e549 orig-commit-author: Deepak Shivamurthy <[email protected]> pr-link: #17104 change-id: cid-551cf01210bb24d39239199adeb398201f67c184
### What changes are proposed in this pull request? Alluxio/Community#643 Fix logging to capture jobControlId ### Why are the changes needed? this will help in getting jobControlId in the logs, and will help in getting status information about the job ### Does this PR introduce any user facing changes? its just log level changes pr-link: Alluxio#16780 change-id: cid-551cf01210bb24d39239199adeb398201f67c184
### What changes are proposed in this pull request? Alluxio/Community#643 Fix logging to capture jobControlId ### Why are the changes needed? this will help in getting jobControlId in the logs, and will help in getting status information about the job ### Does this PR introduce any user facing changes? its just log level changes pr-link: Alluxio#16780 change-id: cid-551cf01210bb24d39239199adeb398201f67c184
What changes are proposed in this pull request?
Alluxio/Community#643
Fix logging to capture jobControlId
Why are the changes needed?
this will help in getting jobControlId in the logs, and will help in getting status information about the job
Does this PR introduce any user facing changes?
its just log level changes