Fredy Wijaya has uploaded a new patch set (#8). ( http://gerrit.cloudera.org:8080/11623 )
Change subject: IMPALA-7673: Support values from other variables in Impala shell --var ...................................................................... IMPALA-7673: Support values from other variables in Impala shell --var Prior to this patch, Impala shell --var could not accept values from other variables unlike the one in Impala interactive shell with the SET command. This patch refactors the logic of variable substitution to use the same logic in both interactive and command line shells. Example: $ impala-shell.sh \ --var="msg1=1" \ --var="msg2=\${var:msg1}2" \ --var="msg3=\${var:msg1}\${var:msg2}" [localhost:21000] default> select ${var:msg3}; Query: select 112 +-----+ | 112 | +-----+ | 112 | +-----+ Testing: - Added a new shell test - Ran all shell tests Change-Id: Ib5b9fda329c45f2e5682f3cbc76d29ceca2e226a --- M shell/impala_shell.py M tests/shell/test_shell_commandline.py 2 files changed, 66 insertions(+), 42 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/23/11623/8 -- To view, visit http://gerrit.cloudera.org:8080/11623 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ib5b9fda329c45f2e5682f3cbc76d29ceca2e226a Gerrit-Change-Number: 11623 Gerrit-PatchSet: 8 Gerrit-Owner: Fredy Wijaya <fwij...@cloudera.com> Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com> Gerrit-Reviewer: Fredy Wijaya <fwij...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Michael Brown <mi...@cloudera.com> Gerrit-Reviewer: Philip Zeyliger <phi...@cloudera.com>