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

Change subject: [cmake] Boost to use /dev/[u]random for UUID generation (take 2)
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/17687/1//COMMIT_MSG
Commit Message:

PS1:
I verified this fix works as expected:
  * Checked the presence of the getrandom symbol in kudu-master binary built in 
release mode: it was there before this fix and it's no longer there with the 
fix.  Checked both for builds on CentOS 8.1 and Centos 7.9
  * Verified that the kudu-master binary built with this patch was able to 
start on CentOS 7 while the binary before this fix failed to start, crashing 
with SIGABRT due to exception thrown from boost: 
'boost::wrapexcept<boost::uuids::entropy_error>' what():  getrandom



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I5eedb644bd9f1cd7a48254580bc8307053ce6c9f
Gerrit-Change-Number: 17687
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Fri, 16 Jul 2021 07:39:48 +0000
Gerrit-HasComments: Yes

Reply via email to