You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@impala.apache.org by jo...@apache.org on 2020/10/23 21:21:03 UTC

[impala] branch master updated: IMPALA-10278: Use full libraries for impalad_executor Docker container

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 <jo...@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 <ta...@cloudera.com>
    Tested-by: Impala Public Jenkins <im...@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