[3/3] kudu git commit: [util] fixed env_util-test.cc compilation on OS X

[alexey]

[util] fixed env_util-test.cc compilation on OS X

The prior version failed to compile on OS X with clang version 3.9.0
(branches/release_39 277311) with error messaged containing:

/opt/local/libexec/llvm-3.9/bin/../include/c++/v1/algorithm:2643:1:
note:
  candidate template ignored: deduced conflicting types for parameter
  '_Tp' ('long' vs. 'long long')
max(const _Tp& __a, const _Tp& __b)
^
/opt/local/libexec/llvm-3.9/bin/../include/c++/v1/algorithm:2653:1:
note:
  candidate template ignored: could not match
    'initializer_list<type-parameter-0-0>' against 'long'
max(initializer_list<_Tp> __t, _Compare __comp)
^

Change-Id: I0ce139c9896a969d51bfcaeabdf6133287c3a897
Reviewed-on: http://gerrit.cloudera.org:8080/6282
Reviewed-by: Mike Percy <mpe...@apache.org>
Tested-by: Kudu Jenkins


Project: http://git-wip-us.apache.org/repos/asf/kudu/repo
Commit: http://git-wip-us.apache.org/repos/asf/kudu/commit/97831ead
Tree: http://git-wip-us.apache.org/repos/asf/kudu/tree/97831ead
Diff: http://git-wip-us.apache.org/repos/asf/kudu/diff/97831ead

Branch: refs/heads/master
Commit: 97831eadf1511449daebe25d3431472b55e3c21c
Parents: 6c2c7c3
Author: Alexey Serbin <aser...@cloudera.com>
Authored: Mon Mar 6 19:43:22 2017 -0800
Committer: Alexey Serbin <aser...@cloudera.com>
Committed: Tue Mar 7 04:28:54 2017 +0000

----------------------------------------------------------------------
 src/kudu/util/env_util-test.cc | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kudu/blob/97831ead/src/kudu/util/env_util-test.cc
----------------------------------------------------------------------
diff --git a/src/kudu/util/env_util-test.cc b/src/kudu/util/env_util-test.cc
index 715f03b..d0a7ae3 100644
--- a/src/kudu/util/env_util-test.cc
+++ b/src/kudu/util/env_util-test.cc
@@ -19,6 +19,7 @@
 #include <sys/time.h>
 #include <unistd.h>
 
+#include <algorithm>
 #include <memory>
 #include <unordered_set>
 
@@ -67,7 +68,7 @@ TEST_F(EnvUtilTest, TestDiskSpaceCheck) {
     ASSERT_OK(env_->GetSpaceInfo(test_dir_, &space_info));
     // Try for 1 less byte than 1% free. This request should be rejected.
     int64_t target_free_bytes = (space_info.capacity_bytes / 100) - 1;
-    int64_t bytes_to_request = std::max(0L, space_info.free_bytes - 
target_free_bytes);
+    int64_t bytes_to_request = std::max<int64_t>(0, space_info.free_bytes - 
target_free_bytes);
     NO_FATALS(AssertNoSpace(VerifySufficientDiskSpace(env_, test_dir_, 
bytes_to_request,
                                                       
kRequestOnePercentReservation)));
   }));