Todd Lipcon has posted comments on this change.

Change subject: Fix CSD and C++11 doc
......................................................................


Patch Set 1: Code-Review+2

(1 comment)

http://gerrit.cloudera.org:8080/#/c/2182/1/docs/installation.adoc
File docs/installation.adoc:

Line 245: the devtoolset will result in `Host compiler appears to require 
libatomic, but cannot find it.`
I think this is fine for this 0.7.0 release, but would be better, I think, to 
just insert a quick check in the top of the build script that verifies a 
sufficiently new compiler is being used and prints a nice error message.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia29cc1dc2f5180ca5c32843aa6c9fedb13ee8b56
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Jean-Daniel Cryans
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Misty Stanley-Jones <[email protected]>
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-HasComments: Yes

Reply via email to