Thomas Tauber-Marshall has posted comments on this change.

Change subject: IMPALA-5207,IMPALA-5214: distcc fixes
......................................................................


Patch Set 2: Code-Review+1

(2 comments)

http://gerrit.cloudera.org:8080/#/c/6655/2//COMMIT_MSG
Commit Message:

Line 9: enable_distcc should reset IMPALA_DISTCC_ENABLED, otherwise 
disable_distcc is not reversible
long line


http://gerrit.cloudera.org:8080/#/c/6655/2/bin/distcc/README.md
File bin/distcc/README.md:

Line 81: 1. Install all required gcc, clang, binutils, etc, versions from the 
toolchain into /opt/Impala-Toolchain.
long line


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic6456d0101cd15287c543cb576be6cd2391f1f26
Gerrit-PatchSet: 2
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Tim Armstrong <tarmstr...@cloudera.com>
Gerrit-Reviewer: Attila Jeges <atti...@cloudera.com>
Gerrit-Reviewer: Thomas Tauber-Marshall <tmarsh...@cloudera.com>
Gerrit-HasComments: Yes

Reply via email to