Fix Kudu hole punch check to work if /tmp is on different fs

/tmp isn't necessarily on the same filesystem as the Kudu data
directory. Fix the check so that it checks the actual Kudu directory.

Change-Id: Ic6aa27569a0650db7dcf5759952cd50c8e47f8c9
Reviewed-on: http://gerrit.cloudera.org:8080/2967
Reviewed-by: Casey Ching <[email protected]>
Tested-by: Internal Jenkins


Project: http://git-wip-us.apache.org/repos/asf/incubator-impala/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-impala/commit/8e64273f
Tree: http://git-wip-us.apache.org/repos/asf/incubator-impala/tree/8e64273f
Diff: http://git-wip-us.apache.org/repos/asf/incubator-impala/diff/8e64273f

Branch: refs/heads/master
Commit: 8e64273fee3869dd8b70c604d3865dc34d752163
Parents: 5454086
Author: Tim Armstrong <[email protected]>
Authored: Thu May 5 12:45:15 2016 -0700
Committer: Tim Armstrong <[email protected]>
Committed: Thu May 12 14:17:56 2016 -0700

----------------------------------------------------------------------
 testdata/cluster/node_templates/cdh5/etc/init.d/kudu-common | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-impala/blob/8e64273f/testdata/cluster/node_templates/cdh5/etc/init.d/kudu-common
----------------------------------------------------------------------
diff --git a/testdata/cluster/node_templates/cdh5/etc/init.d/kudu-common 
b/testdata/cluster/node_templates/cdh5/etc/init.d/kudu-common
index 1ad9189..4bce26a 100644
--- a/testdata/cluster/node_templates/cdh5/etc/init.d/kudu-common
+++ b/testdata/cluster/node_templates/cdh5/etc/init.d/kudu-common
@@ -25,7 +25,7 @@ KUDU_COMMON_ARGS=("-webserver_doc_root=$KUDU_WWW_DIR")
 # identifiable in case the pid file is removed.
 KUDU_COMMON_ARGS+=("-vmodule=IBelongToTheMiniCluster")
 
-HOLE_PUNCH_TEST_FILE=$(mktemp)
+HOLE_PUNCH_TEST_FILE=${NODE_DIR}/var/lib/kudu/hole_punch_test
 if ! fallocate -p -o 1 -l 1 "$HOLE_PUNCH_TEST_FILE" &>/dev/null; then
   KUDU_COMMON_ARGS+=("-block_manager=file")
 fi

Reply via email to