impala-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Adar Dembo (Code Review)" <>
Subject [Impala-ASF-CR] CDH-35265: specify a CM API version in (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

to review the following change.

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

CDH-35265: specify a CM API version in (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/
1 file changed, 4 insertions(+), 1 deletion(-)

  git pull ssh:// refs/changes/36/4136/1
To view, visit
To unsubscribe, visit

Gerrit-MessageType: newchange
Gerrit-Change-Id: I698ce7e9ad860e29313b5c15d9c614a4b1f61904
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Adar Dembo <>
Gerrit-Reviewer: Dimitris Tsirogiannis <>
Gerrit-Reviewer: Matthew Jacobs <>

View raw message