Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/14683 )

Change subject: docs: additional instructions for installing memkind from source
......................................................................


Patch Set 4: Code-Review+2

(1 comment)

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

http://gerrit.cloudera.org:8080/#/c/14683/2//COMMIT_MSG@11
PS2, Line 11: Using the default prefix (/usr/local) means Kudu won't find the 
library.
> I don't know. Personally I view LD_LIBRARY_PATH as a security risk given it
Yep, that would add some extra hustle.  I think installing new version from 
sources sounds like a simpler choice then.



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I7d52d02ad87a27e5fef5ba5c0ab8932128a29762
Gerrit-Change-Number: 14683
Gerrit-PatchSet: 4
Gerrit-Owner: Adar Dembo <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Tue, 12 Nov 2019 08:49:11 +0000
Gerrit-HasComments: Yes

Reply via email to