impala-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Misha Dmitriev (Code Review)" <ger...@cloudera.org>
Subject [Impala-CR](cdh5-trunk) Added support for building Impala Front End separately (and quickly)
Date Wed, 27 Apr 2016 18:28:20 GMT
Misha Dmitriev has posted comments on this change.

Change subject: Added support for building Impala Front End separately (and quickly)
......................................................................


Patch Set 6:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/2875/6/bin/make_impala.sh
File bin/make_impala.sh:

Line 48: fe_only
> Hm, now that I see this I think you were right that the duplication with bu
Yes, I'd rather prefer to finish with this sooner rather than later - I have several other
Cauldron changes all waiting for this one to go in. And I suspect that #1 may result in a
number of side effects to address. So, I would like to get back to #4, and would very much
like to get some guarantee that I won't be asked to change that code again ;)


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I486bb95757334f9df77c4a97150b2b34c5c0e2c4
Gerrit-PatchSet: 6
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Misha Dmitriev <misha@cloudera.com>
Gerrit-Reviewer: Henry Robinson <henry@cloudera.com>
Gerrit-Reviewer: Matthew Jacobs <mj@cloudera.com>
Gerrit-Reviewer: Misha Dmitriev <misha@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <tarmstrong@cloudera.com>
Gerrit-HasComments: Yes

Mime
View raw message