Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16965 )
Change subject: IMPALA-10448: Build impala-profile-tool early for Docker-based tests ...................................................................... IMPALA-10448: Build impala-profile-tool early for Docker-based tests impala-profile-tool is a new dependency for end-to-end tests. The tool is built together with all the other backend tests (so the buildall.sh flag '-notests' can turn off building it), it is actually used in the parallel phase of end-to-end tests. This means a problem for Docker-based builds for the following reasons: - Docker-based tests run BE, FE and various phases of the EE test in separate Docker containers for parallel executions - Test binaries are only built inside the container running BE tests to cut down on the build time and the size of the Docker image that all test containers are based on. - This means that the EE_TEST_PARALLEL container will miss the tool required for running test designed to test it. The solution is to build the tool early, at the end of the build phase running in the build container. There is already another such tool built there (parquet-reader) for similar reason, so just add impala-profile-tool to the same 'make' command there. Tested by running BE_TEST and EE_TEST_PARALLEL phases in a Docker-based build. Change-Id: I60e78ea883f3057c59a345feca38ef08a7f6a0b8 Reviewed-on: http://gerrit.cloudera.org:8080/16965 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M docker/entrypoint.sh 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/16965 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I60e78ea883f3057c59a345feca38ef08a7f6a0b8 Gerrit-Change-Number: 16965 Gerrit-PatchSet: 3 Gerrit-Owner: Laszlo Gaal <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Laszlo Gaal <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]>
