ctubbsii commented on code in PR #4101: URL: https://github.com/apache/accumulo/pull/4101#discussion_r1433579247
########## server/tserver/src/main/java/org/apache/accumulo/tserver/log/DfsLogger.java: ########## @@ -433,11 +427,11 @@ public synchronized void open(String address) throws IOException { byte[] cryptoParams = encrypter.getDecryptionParameters(); CryptoUtils.writeParams(cryptoParams, logFile); - /** - * Always wrap the WAL in a NoFlushOutputStream to prevent extra flushing to HDFS. The - * {@link #write(LogFileKey, LogFileValue)} method will flush crypto data or do nothing when - * crypto is not enabled. - **/ + /* + * Always wrap the WAL in a NoFlushOutputStream to prevent extra flushing to HDFS. The {@link + * #write(LogFileKey, LogFileValue)} method will flush crypto data or do nothing when crypto Review Comment: The `@link` is a javadoc tag. It doesn't do anything in regular comments. This wasn't a valid javadoc in the first place, inside the method, so that change was fine, but it highlights that this javadoc tag can go away too. -- 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. To unsubscribe, e-mail: notifications-unsubscr...@accumulo.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org