Joe McDonnell has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/23024 )

Change subject: IMPALA-13235: Update Opentelemetry C++ SDK Build Options
......................................................................

IMPALA-13235: Update Opentelemetry C++ SDK Build Options

The BUILD_SHARED_LIBS option is now OFF as using the shared library
was forcing that library to be added to LD_LIBRARY_PATH.

The WITH_THREAD_INSTRUMENTATION_PREVIEW option is now ON as this
experimental option provides hooks for logging detailed information
about the internal threads of the OTLP HTTP exporter. Using this
feature requires adding specific code to activate it.

The WITH_OTLP_FILE option is now ON so the static libraries build
with support for exporting telemetry data to a file (which will be
used during Impala testing).

Change-Id: I46c53184204a0feb87b391ff7fe949bdb2812c58
Reviewed-on: http://gerrit.cloudera.org:8080/23024
Reviewed-by: Michael Smith <[email protected]>
Reviewed-by: Joe McDonnell <[email protected]>
Tested-by: Joe McDonnell <[email protected]>
---
M source/opentelemetry-cpp/build.sh
1 file changed, 4 insertions(+), 1 deletion(-)

Approvals:
  Michael Smith: Looks good to me, but someone else must approve
  Joe McDonnell: Looks good to me, approved; Verified

--
To view, visit http://gerrit.cloudera.org:8080/23024
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: native-toolchain
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I46c53184204a0feb87b391ff7fe949bdb2812c58
Gerrit-Change-Number: 23024
Gerrit-PatchSet: 5
Gerrit-Owner: Jason Fehr <[email protected]>
Gerrit-Reviewer: Jason Fehr <[email protected]>
Gerrit-Reviewer: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>

Reply via email to