Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/17354 )
Change subject: [thirdparty] Force Boost to use /dev/random for uuid ...................................................................... Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/17354/2//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/17354/2//COMMIT_MSG@13 PS2, Line 13: In the latest Kudu 1.14.0 binary jar we saw crashes in Hive : pre-commit tests because that jar was compiled on CentOS 7 : and therefore used a getrandom(2) call which was not available : in the Hive precommit container. > Just verified that on one server machine with CentOS 7.3 the kernel version This should fix the Hive pre-commit tests once a new version of the binary jar with this fix is consumed. Unless I am missing something. Regardless I was planning to test the change. We could ensure we compile with a specific Linux version at release time, however the build pipeline where this issue was observed is a more automated thirdparty build. I didn't see much of a downside for this change and I figured a change that made for more transportable binary was good regardless. -- To view, visit http://gerrit.cloudera.org:8080/17354 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I229f691ed851617e2aa183465726b8cc5e10039c Gerrit-Change-Number: 17354 Gerrit-PatchSet: 2 Gerrit-Owner: Grant Henke <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Wed, 28 Apr 2021 13:01:46 +0000 Gerrit-HasComments: Yes
