You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@impala.apache.org by "Zach Amsden (JIRA)" <ji...@apache.org> on 2017/11/22 23:55:00 UTC

[jira] [Closed] (IMPALA-6206) Data loading fails if tests are not built

     [ https://issues.apache.org/jira/browse/IMPALA-6206?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Zach Amsden closed IMPALA-6206.
-------------------------------
       Resolution: Fixed
    Fix Version/s: Impala 2.12.0

IMPALA-6206: Fix data load failure with -notests

When tests are not built, specifically with -notests, instead of
just -skiptests, the be-test target is omitted by cmake, and since
nothing in impalad depends on uda/udf samples, they are not built.
This causes data load to fail on a clean build.

Just build them anyway under the target ImpalaUdf since they
build in a few seconds.

This shaves a few minutes off data load testing by avoiding
the time spent linking tests.  Note: only empirically, not
scientifically measured.

Testing: Run a clean build
  find . -name 'libud[af]sample.so'
  ./be/build/debug/udf_samples/libudasample.so
  ./be/build/debug/udf_samples/libudfsample.so

Change-Id: Idaa193989d77d56b72db05ad54e1fb0a345938fb
Reviewed-on: http://gerrit.cloudera.org:8080/8580
Tested-by: Impala Public Jenkins
Reviewed-by: Zach Amsden <za...@cloudera.com>
---

> Data loading fails if tests are not built
> -----------------------------------------
>
>                 Key: IMPALA-6206
>                 URL: https://issues.apache.org/jira/browse/IMPALA-6206
>             Project: IMPALA
>          Issue Type: Bug
>          Components: Infrastructure
>            Reporter: Zach Amsden
>            Assignee: Zach Amsden
>            Priority: Minor
>             Fix For: Impala 2.12.0
>
>
> When impala is built with -notests, it isn't possible to load data into the minicluster due to missing dependencies - libuda_sample.so and libudf_sample.so don't get built.
> This is unfortunate because linking all the tests takes a non-trivial amount of time.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)