Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12100 )
Change subject: Remove Python 2.4 workarounds in start-impala-cluster.py ...................................................................... Remove Python 2.4 workarounds in start-impala-cluster.py The workarounds are: * Trying to handle the json module not being present. That module was added in Python 2.6 and also isn't used in this file! * Trying to handle import failure of ImpalaCluster. It appears that this is be a workaround for the script being run without the correct PYTHONPATH (set by set-pythonpath.sh) - see IMPALA-7989. This is fixed in this patch by sourcing it in bin/impala-python-common.sh Testing: Ran a build on CentOS 6 with Python 2.4 and confirmed that it was able to start the cluster OK. Reran a build that previously failed with the python import error. Change-Id: I80680804e61fcca81ac2b76604ef8fad70b031f5 Reviewed-on: http://gerrit.cloudera.org:8080/12100 Reviewed-by: Bharath Vissapragada <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M bin/impala-python-common.sh M bin/start-impala-cluster.py 2 files changed, 11 insertions(+), 40 deletions(-) Approvals: Bharath Vissapragada: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/12100 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I80680804e61fcca81ac2b76604ef8fad70b031f5 Gerrit-Change-Number: 12100 Gerrit-PatchSet: 2 Gerrit-Owner: Tim Armstrong <[email protected]> Gerrit-Reviewer: Bharath Vissapragada <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]>
