You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@impala.apache.org by jo...@apache.org on 2022/05/17 02:24:32 UTC

[impala] 01/02: [tools] fix buildall.sh -testdata with prior data

This is an automated email from the ASF dual-hosted git repository.

joemcdonnell pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/impala.git

commit d4cb3afe69e9131ef8784938d231af9b864fbd81
Author: Michael Smith <mi...@cloudera.com>
AuthorDate: Thu May 12 14:06:06 2022 -0700

    [tools] fix buildall.sh -testdata with prior data
    
    The help output for buildall.sh notes running `buildall.sh -testdata` as
    an option to incrementally load test data without formatting the
    mini-cluster. However trying to do that with existing data loaded
    results in an error when running `hadoop fs -mkdir /test-warehouse`. Add
    `-p` so this step is idempotent, allowing the example to work as
    documented.
    
    Change-Id: Icc4ec4bb746abf53f6787fce4db493919806aaa9
    Reviewed-on: http://gerrit.cloudera.org:8080/18522
    Reviewed-by: Quanlong Huang <hu...@gmail.com>
    Tested-by: Impala Public Jenkins <im...@cloudera.com>
---
 testdata/bin/create-load-data.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/testdata/bin/create-load-data.sh b/testdata/bin/create-load-data.sh
index e45295244..12af04b7d 100755
--- a/testdata/bin/create-load-data.sh
+++ b/testdata/bin/create-load-data.sh
@@ -126,7 +126,7 @@ if [[ $SKIP_METADATA_LOAD -eq 0  && "$SNAPSHOT_FILE" = "" ]]; then
   run-step "Generating HBase data" create-hbase.log \
       ${IMPALA_HOME}/testdata/bin/create-hbase.sh
   run-step "Creating /test-warehouse HDFS directory" create-test-warehouse-dir.log \
-      hadoop fs -mkdir /test-warehouse
+      hadoop fs -mkdir -p /test-warehouse
 elif [ $SKIP_SNAPSHOT_LOAD -eq 0 ]; then
   run-step "Loading HDFS data from snapshot: $SNAPSHOT_FILE" \
       load-test-warehouse-snapshot.log \