Fix logging messages in hub/server/src/main/java/alluxio/hub/manager/process/ManagerProcessContext.java #634
Comments
|
/assign @witheredwood |
alluxio-bot
pushed a commit
to Alluxio/alluxio
that referenced
this issue
Feb 28, 2022
### What changes are proposed in this pull request? Fix logging messages in hub/server/src/main/java/alluxio/hub/manager/process/ManagerProcessContext.java ### Why are the changes needed? Fix #634 Alluxio/Community#634 ### Does this PR introduce any user facing changes? None pr-link: #15068 change-id: cid-d8cda1723870cea7a8411055d781350ebe528878
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
https://github.com/Alluxio/alluxio/blob/60102fdc4fbce87b236ddecf483284945fbfd5e0/hub/server/src/main/java/alluxio/hub/manager/process/ManagerProcessContext.java#L393
There are a few wrong references to error in logging, like
LOG.error("Error connecting to hosted hub {}", e);should beLOG.error("Error connecting to hosted hub: ", e);, See more instructions https://docs.alluxio.io/os/user/stable/en/contributor/Code-Conventions.html#error-log-levelThe text was updated successfully, but these errors were encountered: