Alexey Serbin has posted comments on this change. Change subject: env: do not convert all Flush() calls into fsync() on macOS ......................................................................
Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5457/1/src/kudu/util/env_posix.cc File src/kudu/util/env_posix.cc: PS1, Line 653: flags |= SYNC_FILE_RANGE_WAIT_AFTER > Hmm. It's largely academic since FLUSH_SYNC is never used (nor will it be, Ok, I see. Thanks for the clarification. To me, the description of SYNC_FILE_RANGE_WAIT_{BEFORE,AFTER} is a little bit confusing in the manual, but since 'Some details' recommend having both enabled for write-for-data-integrity operations, I also would prefer having both flags specified here. -- To view, visit http://gerrit.cloudera.org:8080/5457 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I01bdd8dbaaad0205c0795a87dd973c8bf0fb87dc Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-HasComments: Yes
