impala-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Skye Wanderman-Milne (Code Review)" <>
Subject [Impala-CR](cdh5-trunk) IMPALA-3132: link ImpalaUdf first in UDF tests
Date Mon, 07 Mar 2016 21:11:20 GMT
Skye Wanderman-Milne has uploaded a new patch set (#2).

Change subject: IMPALA-3132: link ImpalaUdf first in UDF tests

IMPALA-3132: link ImpalaUdf first in UDF tests

ImpalaUdf provides test implementations of MemTracker, FreePool, and
RuntimeState for use with UDF tests. We attempted to link these test
definitions in udf-test, uda-test, and aggregate-functions-test by
replacing the Udf lib dependency with ImpalaUdf. While this caused us
to use the correct compiled version of, the linker would still
use the Runtime implementations of the class methods rather than the
ImpalaUdf definitions, since Runtime case first in the link order.

This patch addresses this by making ImpalaUdf the first linked lib for
UDF tests (and as a result we no longer need to remove the Udf
lib). It also adds calls to InitCommonRuntime() in the affected tests
so their output isn't redirected by default.

Ideally the UDF tests wouldn't need to depend on Runtime at all (or
specifically the parts redefined by ImpalaUdf), but it will require
major refactoring to make this possible.

Change-Id: I619c9e708794aaf1138bdc0087fbe82539c0817e
M be/CMakeLists.txt
M be/src/exprs/
M be/src/udf/
M be/src/udf/
4 files changed, 13 insertions(+), 7 deletions(-)

  git pull ssh:// refs/changes/71/2471/2
To view, visit
To unsubscribe, visit

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I619c9e708794aaf1138bdc0087fbe82539c0817e
Gerrit-PatchSet: 2
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Skye Wanderman-Milne <>
Gerrit-Reviewer: Marcel Kornacker <>
Gerrit-Reviewer: Skye Wanderman-Milne <>

View raw message