elek commented on a change in pull request #234: HDDS-2531. Sonar : remove duplicate string literals in BlockOutputStream URL: https://github.com/apache/hadoop-ozone/pull/234#discussion_r353117304
########## File path: hadoop-hdds/client/src/main/java/org/apache/hadoop/hdds/scm/storage/BlockOutputStream.java ########## @@ -391,26 +391,20 @@ private void watchForCommit(boolean bufferFull) throws IOException { .equals(responseBlockID.getContainerBlockID())); // updates the bcsId of the block blockID.set(responseBlockID); - if (LOG.isDebugEnabled()) { - LOG.debug( - "Adding index " + asyncReply.getLogIndex() + " commitMap size " - + commitWatcher.getCommitInfoMapSize() + " flushLength " - + flushPos + " numBuffers " + byteBufferList.size() - + " blockID " + blockID + " bufferPool size" + bufferPool - .getSize() + " currentBufferIndex " + bufferPool - .getCurrentBufferIndex()); - } + LOG.debug("Adding index {} commitMap size {} flushLength {} " + + "numBuffers {} blockID {} bufferPool size {} " + + "currentBufferIndex {}", asyncReply.getLogIndex(), + commitWatcher.getCommitInfoMapSize(), flushPos, + byteBufferList.size(), blockID, bufferPool.getSize(), + bufferPool.getCurrentBufferIndex()); // for standalone protocol, logIndex will always be 0. commitWatcher .updateCommitInfoMap(asyncReply.getLogIndex(), byteBufferList); } return e; }, responseExecutor).exceptionally(e -> { - if (LOG.isDebugEnabled()) { - LOG.debug( - "putBlock failed for blockID " + blockID + " with exception " + e - .getLocalizedMessage()); - } + LOG.debug("putBlock failed for blockID {} with exception {}", Review comment: This is just the message of the exception not the stack trace itself (as before). I think it's reasonable to keep it as it's just a debug log which helps to localize the problem, the original exception is kept and part of the new exception which is propagated. So the full exception will be printed out somewhere from the callers. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: ozone-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: ozone-issues-h...@hadoop.apache.org