Fredy Wijaya has uploaded a new patch set (#4). ( http://gerrit.cloudera.org:8080/12844 )
Change subject: IMPALA-8340: Rewrite fixes in IMPALA-8317 and IMPALA-8337 ...................................................................... IMPALA-8340: Rewrite fixes in IMPALA-8317 and IMPALA-8337 Fixes in IMPALA-8317 and IMPALA-8337 introduced third-party dependencies in Impala shell which is problematic in multi-Python environment. This patch rewrites the fixes using an alternative solution when dealing with duplicate options without any third-party dependencies. For example: [impala] keyval=msg1=hello,keyval=msg2=world Testing: - Ran all shell tests on Python 2.6 and 2.7. - Ran make_shell_tarball.sh and ran Impala shell from the tarball without any issue. Change-Id: Ifc0bf391ba26cf5a34f622a4157d7287453cc539 --- M infra/python/deps/requirements.txt M shell/option_parser.py M tests/shell/good_impalarc 3 files changed, 8 insertions(+), 28 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/44/12844/4 -- To view, visit http://gerrit.cloudera.org:8080/12844 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ifc0bf391ba26cf5a34f622a4157d7287453cc539 Gerrit-Change-Number: 12844 Gerrit-PatchSet: 4 Gerrit-Owner: Fredy Wijaya <fwij...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
