impala-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ab...@apache.org
Subject incubator-impala git commit: CDH-35265: specify a CM API version in deploy.py (take two)
Date Fri, 26 Aug 2016 23:22:08 GMT
Repository: incubator-impala
Updated Branches:
  refs/heads/master 0e886618e -> 2618766ba


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
Reviewed-on: http://gerrit.cloudera.org:8080/4136
Reviewed-by: Alex Behm <alex.behm@cloudera.com>
Tested-by: Alex Behm <alex.behm@cloudera.com>


Project: http://git-wip-us.apache.org/repos/asf/incubator-impala/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-impala/commit/2618766b
Tree: http://git-wip-us.apache.org/repos/asf/incubator-impala/tree/2618766b
Diff: http://git-wip-us.apache.org/repos/asf/incubator-impala/diff/2618766b

Branch: refs/heads/master
Commit: 2618766baa710d0b92362817b21076e8569c512c
Parents: 0e88661
Author: Adar Dembo <adar@cloudera.com>
Authored: Wed Jan 27 16:53:44 2016 -0800
Committer: Alex Behm <alex.behm@cloudera.com>
Committed: Fri Aug 26 23:21:19 2016 +0000

----------------------------------------------------------------------
 infra/deploy/deploy.py | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-impala/blob/2618766b/infra/deploy/deploy.py
----------------------------------------------------------------------
diff --git a/infra/deploy/deploy.py b/infra/deploy/deploy.py
index 5e26415..c4f40d9 100644
--- a/infra/deploy/deploy.py
+++ b/infra/deploy/deploy.py
@@ -438,7 +438,10 @@ def clone_existing_service(cluster, new_name, template_service):
 
 def main():
     args = parse_args()
-    api = ApiResource(args.host, username=args.user, password=args.password)
+    api = ApiResource(args.host,
+                      username=args.user,
+                      password=args.password,
+                      version=10)
 
     cluster = find_cluster(api, args.cluster)
     check_new_service_does_not_exist(api, cluster, args.service_name)


Mime
View raw message