impala-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Adar Dembo (Code Review)" <ger...@cloudera.org>
Subject [Impala-ASF-CR] CDH-35265: specify a CM API version in deploy.py (take two)
Date Fri, 26 Aug 2016 21:00:19 GMT
Hello Matthew Jacobs, Dimitris Tsirogiannis,

I'd like you to do a code review.  Please visit

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

to review the following change.

Change subject: CDH-35265: specify a CM API version in deploy.py (take two)
......................................................................

CDH-35265: specify a CM API version in deploy.py (take two)

This commit was accidentally reverted when Kimpala merged to impala trunk.
Here's a cherry-pick to restore it. Original commit message (minus gerrit
metadata) follows.

Otherwise we'll get the latest in the local copy of cm_api, which may be
greater than what the CM server supports. I've chosen API version 10 here;
we can probably get by with older versions, but this one is supported by CM
5.4.3, which is the oldest CM to support Kudu.

I tested this by cloning an Impala service in a cluster managed by CM 5.4.3
and running CDH 5.4.10. The service started and was running from the
correct parcel.

Change-Id: I698ce7e9ad860e29313b5c15d9c614a4b1f61904
---
M infra/deploy/deploy.py
1 file changed, 4 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/36/4136/1
-- 
To view, visit http://gerrit.cloudera.org:8080/4136
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I698ce7e9ad860e29313b5c15d9c614a4b1f61904
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Adar Dembo <adar@cloudera.com>
Gerrit-Reviewer: Dimitris Tsirogiannis <dtsirogiannis@cloudera.com>
Gerrit-Reviewer: Matthew Jacobs <mj@cloudera.com>

Mime
View raw message