[Impala-CR](cdh5-trunk) IMPALA-3132: link ImpalaUdf first in UDF tests

["Skye Wanderman-Milne (Code Review)" <ge...@cloudera.org>]

Skye Wanderman-Milne has uploaded a new change for review.

  http://gerrit.cloudera.org:8080/2471

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 udf.cc, 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/aggregate-functions-test.cc
M be/src/udf/uda-test.cc
M be/src/udf/udf-test.cc
4 files changed, 11 insertions(+), 7 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala refs/changes/71/2471/1
-- 
To view, visit http://gerrit.cloudera.org:8080/2471
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I619c9e708794aaf1138bdc0087fbe82539c0817e
Gerrit-PatchSet: 1
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Skye Wanderman-Milne <sk...@cloudera.com>