Adar Dembo has posted comments on this change.

Change subject: IMPALA-3449: Kudu deploy.py should find clusters by displayName
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/3106/1/infra/deploy/deploy.py
File infra/deploy/deploy.py:

Line 105:       return api.get_cluster(cluster_name)
Nit: regardless of whether we use an explicit cluster or an implicit one, can 
we print out its name (display name is fine) when we choose it? I've found that 
to be helpful when debugging the script.


-- 
To view, visit http://gerrit.cloudera.org:8080/3106
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia5b828e3f32c5483e76b055a5fcd0487a54ee9ea
Gerrit-PatchSet: 1
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Casey Ching <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-HasComments: Yes

Reply via email to