Tim Armstrong has uploaded a new change for review.

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

Change subject: IMPALA-5027: make udf headers buildable externally
......................................................................

IMPALA-5027: make udf headers buildable externally

This fixes a number of issues that prevented building it:
* Use of C++11 features
* Use of internal headers that are only available in the Impala source
  tree

Testing:
Copied the headers and libImpalaUdf.a to /usr and confirmed I could
build impala-udf-samples (with some modifications to udf-test-harness
to use some modified arguments to the test harness).

We really need a better solution for automatically testing this, e.g.
IMPALA-5026, but I wanted to fix the known bugs first.

Change-Id: Ibdcdcd96c953cbabc3de04ca66d7aa1774ada99e
---
M be/src/udf/uda-test-harness-impl.h
M be/src/udf/uda-test-harness.h
M be/src/udf/udf-debug.h
M be/src/udf/udf-test-harness.h
M be/src/udf/udf.h
5 files changed, 45 insertions(+), 15 deletions(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ibdcdcd96c953cbabc3de04ca66d7aa1774ada99e
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Tim Armstrong <[email protected]>

Reply via email to