dlmarion commented on code in PR #3076:
URL: https://github.com/apache/accumulo/pull/3076#discussion_r1021932231


##########
server/tserver/src/main/java/org/apache/accumulo/tserver/log/DfsLogger.java:
##########
@@ -417,6 +417,13 @@ public synchronized void open(String address) throws 
IOException {
       else
         logFile = fs.create(logfilePath, true, 0, replication, blockSize);
 
+      // Tell the DataNode that the write ahead log does not need to be cached 
in the OS page cache
+      try {
+        logFile.setDropBehind(Boolean.TRUE);

Review Comment:
   Javadoc says that if parameter is null, then it defaults to configuration, 
which should be the value of `dfs.datanode.drop.cache.behind.writes`



-- 
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]

Reply via email to