Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12078 )
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's less clear why this was necessary, but I can't see a reason why we wouldn't be able to import our own code that is used in many places in the tests. Neither seems necessary at this point. Testing: Ran a build on CentOS 6 with Python 2.4 and confirmed that it was able to start the cluster OK. Change-Id: Ia28a7b75f2e2804834e5df57ef7998b360112e5c Reviewed-on: http://gerrit.cloudera.org:8080/12078 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M bin/start-impala-cluster.py 1 file changed, 9 insertions(+), 40 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/12078 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ia28a7b75f2e2804834e5df57ef7998b360112e5c Gerrit-Change-Number: 12078 Gerrit-PatchSet: 4 Gerrit-Owner: Tim Armstrong <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Lars Volker <[email protected]> Gerrit-Reviewer: Philip Zeyliger <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]>
