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
[SMALLFIX] Fix LOG.warn without context #6924
Conversation
365039d
to
661cc8f
Compare
Merged build finished. Test FAILed. |
1 similar comment
Merged build finished. Test FAILed. |
661cc8f
to
85d8dda
Compare
Merged build finished. Test PASSed. |
Test PASSed. |
Automated checks report:
All checks passed! |
@gpang PTAL |
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.
I just left a minor comment.
LGTM
@@ -1504,7 +1504,7 @@ private void deleteFromEntry(DeleteFileEntry entry) { | |||
failedToDelete = !ufsDeleter.delete(alluxioUriToDel, delInode); | |||
} | |||
} catch (InvalidPathException e) { | |||
LOG.warn(e.getMessage()); | |||
LOG.warn("Failed to delete path from UFS: {}", e.getMessage()); |
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.
Will e.getMessage()
already include the UFS path?
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.
This exception is thrown when ufsDeleter failed to resolve mount table as the alluxio URL is invalid.
thus, it is about alluxio URL which will be included in exception message
Logging messages from
LOG.warn(e.getMessage());
will not be able to locate.