Henry Robinson has posted comments on this change. Change subject: IMPALA-4931: Update squeasel to include patch to disable SSL init ......................................................................
Patch Set 1: Code-Review+2 That compiler define is no longer relevant - it was removed from Squeasel in https://github.com/cloudera/squeasel/commit/2404f850003e8124e3f7d6340e861595163b3681. It used to control whether or not Squeasel would use dlopen to find OpenSSL symbols, but I think we statically link OpenSSL now. Regardless, NO_SSL_DL isn't in Impala's squeasel.c, so it doesn't have any effect right now. -- To view, visit http://gerrit.cloudera.org:8080/6006 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3d28f2a5391b8a7d39a50002ff1d96ef3d927567 Gerrit-PatchSet: 1 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Matthew Jacobs <[email protected]> Gerrit-Reviewer: Henry Robinson <[email protected]> Gerrit-Reviewer: Matthew Jacobs <[email protected]> Gerrit-Reviewer: Sailesh Mukil <[email protected]> Gerrit-HasComments: No
