[Public] Default behavior is not changed. Thank you, Josh for your appreciation. This is my first patch, and it means lot to me.
Thanks again, Amit +1 to adding the feature, clear and easy configurability, and if after a major cycle we can say with confidence it's beating the status quo in the vast majority of general cases, flip default. I mean, logically it should be, but infra software at the scale we do requires great care. :) This is great work Amit - well done. On Mon, Oct 16, 2023, at 4:28 PM, Dinesh Joshi wrote: I haven't looked at the patch yet so take whatever I say here with a pinch of salt. Philosophically, defaults should not change unless there is a clear demonstrable benefit in majority cases for our users. In this case DirectIO should have clear benefits. That said, this is a new feature and I would personally default it to off. We should document it and allow for our users to enable it. This derisks the project in case there is an inadvertent change in behavior. Dinesh On Oct 15, 2023, at 11:34 PM, Pawar, Amit <amit.pa...@amd.com<mailto:amit.pa...@amd.com>> wrote: [Public] Hi, CommitLog uses mmap (memory mapped ) segments by default. Direct-IO feature is proposed through new PR[1] to improve the CommitLog IO speed. Enabling this by default could be useful feature to address IO bottleneck seen during peak load. Need your input regarding changing this default. Please suggest. https://issues.apache.org/jira/browse/CASSANDRA-18464 thanks, Amit Pawar [1] - https://github.com/apache/cassandra/pull/2777