[ https://issues.apache.org/jira/browse/HADOOP-16431?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16890230#comment-16890230 ]
Lisheng Sun edited comment on HADOOP-16431 at 7/22/19 3:07 PM: --------------------------------------------------------------- hi [~elgoiri] [~ayushtkn] [~xkrogen]. Could you have time to review this patch? Thank you. was (Author: leosun08): hi [~elgoiri] [~ayushtkn], Could you have time to review this patch? Thank you. > Remove useless log in IOUtils.java and ExceptionDiags.java > ---------------------------------------------------------- > > Key: HADOOP-16431 > URL: https://issues.apache.org/jira/browse/HADOOP-16431 > Project: Hadoop Common > Issue Type: Improvement > Reporter: Lisheng Sun > Priority: Minor > Attachments: HADOOP-16431.001.patch, HADOOP-16431.002.patch > > > When there is no String Constructor for the exception, we Log a Warn Message, > and rethrow the exception. We can change the Log level to TRACE/DEBUG. > {code:java} > private static <T extends IOException> T wrapWithMessage( > T exception, String msg) { > Class<? extends Throwable> clazz = exception.getClass(); > try { > Constructor<? extends Throwable> ctor = > clazz.getConstructor(String.class); > Throwable t = ctor.newInstance(msg); > return (T) (t.initCause(exception)); > } catch (Throwable e) { > LOG.trace("Unable to wrap exception of type " + > clazz + ": it has no (String) constructor", e); > return exception; > } > } > {code} -- This message was sent by Atlassian JIRA (v7.6.14#76016) --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org