This is an automated email from the ASF dual-hosted git repository. michaelsmith pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/impala.git
commit 9605be3fed92cdc7c37f02afa7466890aa0064a9 Author: Michael Smith <[email protected]> AuthorDate: Tue Jun 28 10:21:37 2022 -0700 [tools] Comment out empty export Startup scripts require that IMPALA_CLUSTER_MAX_MEM_GB be set to a valid value, or not set. In zsh, calling `export FOO` twice results in setting the variable to an empty string (arguably a bug), so sourcing impala-config.sh is non-idempotent. Comments out `export IMPALA_CLUSTER_MAX_MEM_GB` so it won't be marked for export and updates it to provide clearer documentation. Change-Id: I288281704480e1564a27984d5e23a54fd39b5570 Reviewed-on: http://gerrit.cloudera.org:8080/18677 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Michael Smith <[email protected]> --- bin/impala-config.sh | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/bin/impala-config.sh b/bin/impala-config.sh index 5f788c37b..2cf2ee2f2 100755 --- a/bin/impala-config.sh +++ b/bin/impala-config.sh @@ -129,8 +129,9 @@ unset IMPALA_LLVM_URL export IMPALA_LLVM_ASAN_VERSION=5.0.1-p5 unset IMPALA_LLVM_ASAN_URL -# Maximum memory available for mini-cluster and CDH cluster -export IMPALA_CLUSTER_MAX_MEM_GB +# To limit maximum memory available for the mini-cluster and CDH cluster, add the +# following in $IMPALA_HOME/bin/impala-config-local.sh +# export IMPALA_CLUSTER_MAX_MEM_GB=<value> # LLVM stores some files in subdirectories that are named after what # version it thinks it is. We might think it is 5.0.1-p1, based on a
