Hello Michael Ho, Impala Public Jenkins,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/10762
to review the following change.
Change subject: IMPALA-5873: Check for existence of sync_file_range()
......................................................................
IMPALA-5873: Check for existence of sync_file_range()
On some legacy platforms, the system call sync_file_range()
is not defined. Some Kudu code calls it unconditionally
on Linux platforms. Let's check for its existence and use
the workaround already defined for non-Linux platforms.
Testing done: ran packaging build on Centos55. Failed consistently before.
Change-Id: Ib4f9409ca3a519b8b9eb5ca91a3c2a6c7c25e34f
Reviewed-on: http://gerrit.cloudera.org:8080/7923
Reviewed-by: Michael Ho <[email protected]>
Tested-by: Impala Public Jenkins
---
M be/src/kudu/util/env_posix.cc
1 file changed, 2 insertions(+), 2 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/62/10762/1
--
To view, visit http://gerrit.cloudera.org:8080/10762
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib4f9409ca3a519b8b9eb5ca91a3c2a6c7c25e34f
Gerrit-Change-Number: 10762
Gerrit-PatchSet: 1
Gerrit-Owner: Lars Volker <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Michael Ho <[email protected]>
