[ https://issues.apache.org/jira/browse/HDFS-14125?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16755454#comment-16755454 ]
Shweta commented on HDFS-14125: ------------------------------- Sure. I was actually referring to using the {} in the String.format() itself, should have been more clear in my previous comment. Using %s and %d as you mention seems reasonable as you are dealing with String and digits. Looks good to me +1 (non-binding). > Use parameterized log format in ECTopologyVerifier > -------------------------------------------------- > > Key: HDFS-14125 > URL: https://issues.apache.org/jira/browse/HDFS-14125 > Project: Hadoop HDFS > Issue Type: Improvement > Components: erasure-coding > Affects Versions: 3.3.0 > Reporter: Kitti Nanasi > Assignee: Kitti Nanasi > Priority: Trivial > Attachments: HDFS-14125.001.patch > > > ECTopologyVerifier introduced in > [HDFS-12946|https://issues.apache.org/jira/browse/HDFS-12946] should use a > parameterized log format. -- This message was sent by Atlassian JIRA (v7.6.3#76005) --------------------------------------------------------------------- To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org