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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
