[ 
https://issues.apache.org/jira/browse/IMPALA-2764?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18116214#comment-18116214
 ] 

ASF subversion and git services commented on IMPALA-2764:
---------------------------------------------------------

Commit a51ab1940108c3be9a5564b5045fe86773399731 in impala's branch 
refs/heads/master from Michael Smith
[ https://gitbox.apache.org/repos/asf?p=impala.git;h=a51ab1940 ]

IMPALA-15362: Remove dead code after IMPALA-2764

Removes undef.cpp, which was only needed when libsasl2 was statically
linked. After IMPALA-2764 it's dynamically linked and undef.cpp is no
longer necessary.

Fixes thrift-server-test with the 'mold' linker, where the symbols
in undef.cpp would cause conflicts.

Change-Id: I2ec2bb9854151f1177e7022cd409733564673675
Reviewed-on: http://gerrit.cloudera.org:8080/24866
Reviewed-by: Csaba Ringhofer <[email protected]>
Reviewed-by: Alexey Serbin <[email protected]>
Tested-by: Michael Smith <[email protected]>


> Impala should not statically link Cyrus SASL
> --------------------------------------------
>
>                 Key: IMPALA-2764
>                 URL: https://issues.apache.org/jira/browse/IMPALA-2764
>             Project: IMPALA
>          Issue Type: Bug
>          Components: Infrastructure
>    Affects Versions: Impala 2.3.0
>            Reporter: Martin Grund
>            Assignee: Martin Grund
>            Priority: Major
>             Fix For: Impala 2.5.0
>
>
> Impala should not statically link to Cyrus SASL for some reasons:
>   * Impala links the library static, but the auth plugins will be loaded 
> dynamically and may come from a different version
>   * If SASL is installed on the target host in a non-standard location, 
> Impala will fail to be able to use SASL and simply stop working.
>   * Impala should not ship libraries that may receive security updated more 
> frequently than Impala is released (e.g. OpenSSL).



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to