Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/21282 )
Change subject: [CMakeLists] Make kudu_test_main static ...................................................................... [CMakeLists] Make kudu_test_main static This change makes sure that kudu_util.so is loaded at the beginning of the library search order. This is needed because in unwind_safeness.cc there is a dlsym execution in the constructor function, that creates a wrapper for dlopen and dlclose to prevent a potential deadlock during unwind stack resolve. It looks for the next declaration of the functions called "dlopen" and "dlclose" in shared object files. If kudu_util is loaded too late, then it won't find these functions and throws an error. This happens in ubuntu 22.04 test runs. To solve this issue, the kudu_test_main was changed to a static library and kudu_util was moved to the front of the library list. This is a best effort fix, and it should only have impact on test execution. Change-Id: I0dfeb1fa04ed91e95fd1f8d789f020dd44289fea Reviewed-on: http://gerrit.cloudera.org:8080/21282 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Alexey Serbin <[email protected]> --- M src/kudu/util/CMakeLists.txt 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Alexey Serbin: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/21282 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I0dfeb1fa04ed91e95fd1f8d789f020dd44289fea Gerrit-Change-Number: 21282 Gerrit-PatchSet: 8 Gerrit-Owner: Ádám Bakai <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Zoltan Martonka <[email protected]> Gerrit-Reviewer: Ádám Bakai <[email protected]>
