[impala] branch master updated (b867f4c4f -> 298d064d2)

[jo...@apache.org]

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

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


    from b867f4c4f IMPALA-10745 (part 2): Support Kerberos over HTTP for impala-shell
     new d4cb3afe6 [tools] fix buildall.sh -testdata with prior data
     new 298d064d2 [tools] fix set-classpath for other POSIX shells

The 2 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 bin/set-classpath.sh             | 2 +-
 testdata/bin/create-load-data.sh | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

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

[jo...@apache.org]

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 \

[impala] 02/02: [tools] fix set-classpath for other POSIX shells

[jo...@apache.org]

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 298d064d23b9fc1eb10637406e92809f3f994e92
Author: Michael Smith <mi...@cloudera.com>
AuthorDate: Wed May 11 13:28:35 2022 -0700

    [tools] fix set-classpath for other POSIX shells
    
    Updates comparison to use `=` instead of `==` so it works with other
    POSIX shells like zsh. `==` is a bash-specific alias for `=`.
    
    Change-Id: Id09ab96a6b1de7cd96eb739221d6aeaa239888d1
    Reviewed-on: http://gerrit.cloudera.org:8080/18519
    Reviewed-by: Impala Public Jenkins <im...@cloudera.com>
    Tested-by: Impala Public Jenkins <im...@cloudera.com>
---
 bin/set-classpath.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/bin/set-classpath.sh b/bin/set-classpath.sh
index d15bc00aa..bbdc214bd 100644
--- a/bin/set-classpath.sh
+++ b/bin/set-classpath.sh
@@ -48,7 +48,7 @@ fi
 
 CLASSPATH=$(cat $FE_CP_FILE):"$CLASSPATH"
 
-if [ "${1:-notest}" == "test" ]; then
+if [[ "${1:-notest}" = "test" ]]; then
   FE_TEST_CP_FILE="$IMPALA_HOME/fe/target/test-classpath.txt"
   CLASSPATH=$(cat $FE_TEST_CP_FILE):"$CLASSPATH"
 fi