Alexey Serbin has uploaded this change for review. (
http://gerrit.cloudera.org:8080/24963
Change subject: [util] an update on --never_fsync flag usage
......................................................................
[util] an update on --never_fsync flag usage
This changelist extends --never_fsync flag's application into the
Posix{RW,Writrable}File::Flush() methods: they invoke sync_file_range()
system call, conceptually doing exactly the same what fsync() does for
a portion of a file [1]. The motivation for this came from experiments
of running a test workload generated by the `kudu perf loadgen` CLI tool
against a Kudu cluster that doesn't synchronize data to the underlying
storage at critical data points.
Also, wrap all FLAGS_never_fsync-based criteria into the EXPECT_FALSE()
macro since --never_fsync is a hidden/test-only flag which shouldn't be
ever enabled in a real production system.
[1] https://man7.org/linux/man-pages/man2/sync_file_range.2.html
Change-Id: I637e3bd98da98495a38f78176a4a862ab6ae8dd3
---
M src/kudu/util/env_posix.cc
1 file changed, 13 insertions(+), 3 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/63/24963/1
--
To view, visit http://gerrit.cloudera.org:8080/24963
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I637e3bd98da98495a38f78176a4a862ab6ae8dd3
Gerrit-Change-Number: 24963
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <[email protected]>
