This is an automated email from the ASF dual-hosted git repository. joemcdonnell pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/impala.git
The following commit(s) were added to refs/heads/master by this push: new cfa8a7a IMPALA-10278: Use full libraries for impalad_executor Docker container cfa8a7a is described below commit cfa8a7a5e54fe7826945f688df03eabc3cd03dd4 Author: Joe McDonnell <joemcdonn...@cloudera.com> AuthorDate: Fri Oct 23 08:53:11 2020 -0700 IMPALA-10278: Use full libraries for impalad_executor Docker container This backs out the piece of IMPALA-10016 that used a pared-down set of libraries for the impalad_executor. That pared-down set was missing org.apache.impala.common.JniUtil, which prevented the impalad_executor container from starting up. Testing: - Ran a docker core job with one coord_exec and two executors and it was able to startup where it wouldn't before Change-Id: Ieecca61cd3c11f446b922a04fdeb5fd0c90fc971 Reviewed-on: http://gerrit.cloudera.org:8080/16640 Reviewed-by: Tim Armstrong <tarmstr...@cloudera.com> Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> --- docker/impalad_executor/Dockerfile | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/docker/impalad_executor/Dockerfile b/docker/impalad_executor/Dockerfile index e844ab0..74d8204 100644 --- a/docker/impalad_executor/Dockerfile +++ b/docker/impalad_executor/Dockerfile @@ -18,8 +18,10 @@ ARG BASE_IMAGE=impala_base FROM ${BASE_IMAGE} -# Only copy library dependencies required to run Impala executors. -COPY --chown=impala exec-lib /opt/impala/lib +# IMPALA-10278: Temporarily switch back to including all the same libraries +# as the coordinator. This will require further changes to exec-lib to be +# able to use it here. +COPY --chown=impala lib /opt/impala/lib # Externally-facing ports # Debug webserver