Casey Ching has posted comments on this change. Change subject: Address last few review items on the impala-kudu merge ......................................................................
Patch Set 1: (1 comment) I'm still not so sure about keeping deploy.py. Do you happen to know if it's the same as https://github.com/cloudera/impala-kudu/blob/feature/kudu/infra/deploy/deploy.py ? Seems bad to have two of them. http://gerrit.cloudera.org:8080/#/c/2545/1/infra/deploy/deploy.py File infra/deploy/deploy.py: Line 3: # Confidential Cloudera Information: Covered by NDA. This line should be removed. -- To view, visit http://gerrit.cloudera.org:8080/2545 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ibc4b30180a8f23394c7afc32b32668b05f142eff Gerrit-PatchSet: 1 Gerrit-Project: Impala Gerrit-Branch: cdh5-trunk Gerrit-Owner: David Ribeiro Alves <[email protected]> Gerrit-Reviewer: Casey Ching <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]> Gerrit-HasComments: Yes
