Aleksandr Efimov has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/24358


Change subject: IMPALA-12955: Slim down impala-profile-tool dependencies
......................................................................

IMPALA-12955: Slim down impala-profile-tool dependencies

impala-profile-tool currently links against the broad backend dependency
set used by other utility binaries. That pulls in runtime dependencies
that the standalone profile reader does not need, including JVM, HDFS,
security, and RPC-related libraries.

This patch gives impala-profile-tool a narrower set of link inputs.
RuntimeProfile, Status, and ErrorMsg implementation code is split at
object-file boundaries so the tool can keep profile parsing and printing
without linking unrelated backend helpers.

The archive profile decode path uses zlib and compact Thrift directly
to avoid the heavier Codec/MemPool/thrift-util stack for the standalone
tool.

A Linux-only regression test checks that known unused runtime
dependencies do not reappear in the tool's dynamic section.

Tests:
- make -C be/src/util impala-profile-tool
- make -C be/src/common Common
- make -C be/src/util Util
- readelf -d be/build/debug/util/impala-profile-tool | grep NEEDED
- bin/impala-python3 -m pytest \
  tests/observability/test_profile_tool.py -q
- git diff --check

Change-Id: Id476411dcf6c46079e2e2addc5cdd552bc23f8a1
---
M be/src/common/CMakeLists.txt
A be/src/common/status-core.cc
M be/src/common/status.cc
M be/src/runtime/mem-pool.h
A be/src/runtime/summary-stats.h
M be/src/util/CMakeLists.txt
A be/src/util/error-msg.cc
M be/src/util/error-util.cc
A be/src/util/runtime-profile-archive.cc
A be/src/util/runtime-profile-periodic.cc
A be/src/util/runtime-profile-serialization.cc
M be/src/util/runtime-profile.cc
M tests/observability/test_profile_tool.py
13 files changed, 770 insertions(+), 446 deletions(-)



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Id476411dcf6c46079e2e2addc5cdd552bc23f8a1
Gerrit-Change-Number: 24358
Gerrit-PatchSet: 1
Gerrit-Owner: Aleksandr Efimov <[email protected]>

Reply via email to