Hello Lars Volker, Impala Public Jenkins,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/12078

to look at the new patch set (#2).

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
---
M bin/start-impala-cluster.py
1 file changed, 9 insertions(+), 40 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/78/12078/2
-- 
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: newpatchset
Gerrit-Change-Id: Ia28a7b75f2e2804834e5df57ef7998b360112e5c
Gerrit-Change-Number: 12078
Gerrit-PatchSet: 2
Gerrit-Owner: Tim Armstrong <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Lars Volker <[email protected]>

Reply via email to