blambov commented on code in PR #2777:
URL: https://github.com/apache/cassandra/pull/2777#discussion_r1371737336


##########
conf/cassandra.yaml:
##########
@@ -589,6 +589,10 @@ commitlog_segment_size: 32MiB
 #     parameters:
 #         -
 
+# Configure commitlog disk access mode preferred order:direct_io, mmap, 
standard.
+# Default access mode is mmap. Direct I/O uses minimum page size to flush to 
disk.
+commitlog_disk_access_mode : mmap

Review Comment:
   My personal preference would be not only to change the default, but to scrap 
memory-mapped logs altogether. However,  for things like these we can't change 
the default unless everyone agrees or there's no expected downside. In this 
case a downside is the fact that a memory-mapped log might effectively persist 
data more often in the default periodic mode.
   
   `mmap` should thus remain default, with a switch to `direct` as one of the 
items in the optimized configuration envisioned by CASSANDRA-18753.



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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to