uros-b commented on code in PR #56782:
URL: https://github.com/apache/spark/pull/56782#discussion_r3476927720
##########
resource-managers/kubernetes/docker/src/main/dockerfiles/spark/Dockerfile:
##########
@@ -36,7 +36,8 @@ ARG spark_uid=185
RUN set -ex && \
apt-get update && \
ln -s /lib /lib64 && \
- apt install -y --no-install-recommends bash tini libc6 libpam-modules
krb5-user libnss3 procps net-tools logrotate libssl-dev && \
+ apt install -y --no-install-recommends bash tini libc6 libjemalloc2
libpam-modules krb5-user libnss3 procps net-tools logrotate libssl-dev && \
+ ln -s "$(find /usr/lib -name 'libjemalloc.so.2' | head -n1)"
/usr/lib/libjemalloc.so.2 && \
Review Comment:
Nit: the symlink may be redundant. The package installs the lib into
/usr/lib/.../, which is already on the default dynamic-linker search path and
registered in the ldconfig cache. So LD_PRELOAD=libjemalloc.so.2 (soname only,
no path) would resolve on both arches without any symlink?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
