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 <[email protected]>
Gerrit-Reviewer: Henry Robinson <[email protected]>
Gerrit-Reviewer: Matthew Jacobs <[email protected]>
Gerrit-Reviewer: Misha Dmitriev <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-HasComments: Yes

Reply via email to