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

Change subject: KUDU-2990: remove memkind/libnuma and dynamically link memkind 
at runtime
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/14620/1/src/kudu/util/nvm_cache.cc
File src/kudu/util/nvm_cache.cc:

http://gerrit.cloudera.org:8080/#/c/14620/1/src/kudu/util/nvm_cache.cc@138
PS1, Line 138: RTLD_LAZY
Maybe, use RTLD_NOW instead to detect possible issues with other symbols 
required (if any)?



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I4f474196aa98b5fa6e5966b9a3aea9a7e466805c
Gerrit-Change-Number: 14620
Gerrit-PatchSet: 1
Gerrit-Owner: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: ye yuqiang <yuqiang...@intel.com>
Gerrit-Comment-Date: Sun, 03 Nov 2019 08:02:28 +0000
Gerrit-HasComments: Yes

Reply via email to