Surya Hebbar has uploaded a new patch set (#10). ( 
http://gerrit.cloudera.org:8080/23154 )

Change subject: IMPALA-9846: Enable Aggregated Runtime Profile by Default
......................................................................

IMPALA-9846: Enable Aggregated Runtime Profile by Default

The traditional query profile generated bigger forests of child counters
and runtime profiles, from fragments and operators upto instance levels.

In the aggregated profile, the depth of the forests have been reduced
by transforming instance-level counters into operator level arrays
or maps.

With this change, aggregated runtime profile is enabled by having
'gen_aggregated_profile' set to 'true' by default. This serves as
replacement for 'gen_experimental_profile'.

The following tests are to be modified to support the usage of
aggregated runtime profile -

Backend -
No further changes are required for backend tests.

Frontend -
org.apache.impala.customcluster.ExternalFrontendTest

  - testExternalFrontendBinary

  - testExternalFrontendHttp

End-to-end Python tests  -
(partial/all child tests of the following)

  custom_cluster
    - test_admission_controller.*

    - test_ext_data_sources.*

    - test_iceberg_strict_data.*

    - test_mt_dop.*

    - test_query_live.*

    - test_query_log.*

    - test_runtime_filter_aggregation.*

    - test_tuple_cache.*

  query_test
    - test_acid.*

    - test_aggregation.*

    - test_analytic_tpcds.*

    - test_codegen.*

    - test_hash_join_timer.*

    - test_iceberg.*

    - test_insert.*

    - test_join_queries.*

    - test_kudu.*

    - test_mem_usage_scaling.*

    - test_mt_dop.*

    - test_nested_types.*

    - test_observability.*

    - test_parquet_late_materialization.*

    - test_partitioning.*

    - test_queries.*

    - test_result_spooling.*

    - test_scanners.*

    - test_runtime_filters.*

    - test_tpch_queries.*

Also, in aggregated profile, instance-level time series counters
(i.e. MemoryUsage, ThreadUsage) are currently not being included,
as they are considered too large for longer queries.

Further sampling or aggregation(IMPALA-14256) is required to include
these counters.

Change-Id: If41d6322361fba82c946efd614cc7d28cb1c36e8
---
M be/src/runtime/coordinator-backend-state.cc
M be/src/runtime/coordinator-backend-state.h
M be/src/runtime/coordinator.cc
M be/src/service/impala-server.cc
M be/src/service/query-state-record.cc
M be/src/util/runtime-profile-counters.h
M be/src/util/runtime-profile-test.cc
M be/src/util/runtime-profile.cc
M be/src/util/runtime-profile.h
M testdata/impala-profiles/README
M testdata/impala-profiles/impala_profile_log_tpcds_compute_stats.expected.json
M 
testdata/impala-profiles/impala_profile_log_tpcds_compute_stats.expected.pretty.json
M testdata/impala-profiles/impala_profile_log_tpcds_compute_stats.expected.txt
M 
testdata/impala-profiles/impala_profile_log_tpcds_compute_stats_default.expected.txt
M 
testdata/impala-profiles/impala_profile_log_tpcds_compute_stats_extended.expected.pretty.json
M 
testdata/impala-profiles/impala_profile_log_tpcds_compute_stats_extended.expected.txt
M 
testdata/impala-profiles/impala_profile_log_tpcds_compute_stats_v2.expected.json
M 
testdata/impala-profiles/impala_profile_log_tpcds_compute_stats_v2_default.expected.txt
M 
testdata/impala-profiles/impala_profile_log_tpcds_compute_stats_v2_extended.expected.pretty.json
M 
testdata/impala-profiles/impala_profile_log_tpcds_compute_stats_v2_extended.expected.txt
M testdata/workloads/functional-query/queries/QueryTest/binary-type.test
M 
testdata/workloads/functional-query/queries/QueryTest/compute-stats-incremental.test
M testdata/workloads/functional-query/queries/QueryTest/compute-stats.test
M testdata/workloads/functional-query/queries/QueryTest/exprs.test
M 
testdata/workloads/functional-query/queries/QueryTest/full-acid-original-file.test
M 
testdata/workloads/functional-query/queries/QueryTest/iceberg-compute-stats.test
M 
testdata/workloads/functional-query/queries/QueryTest/iceberg-scan-metrics-with-deletes.test
M 
testdata/workloads/functional-query/queries/QueryTest/impala-ext-jdbc-tables.test
M testdata/workloads/functional-query/queries/QueryTest/in_list_filters.test
M testdata/workloads/functional-query/queries/QueryTest/insert.test
M testdata/workloads/functional-query/queries/QueryTest/insert_null.test
M 
testdata/workloads/functional-query/queries/QueryTest/kudu_insert_mem_limit.test
M 
testdata/workloads/functional-query/queries/QueryTest/nested-types-scanner-array-materialization.test
M testdata/workloads/functional-query/queries/QueryTest/nested-types-tpch.test
M 
testdata/workloads/functional-query/queries/QueryTest/parquet-corrupt-rle-counts.test
M 
testdata/workloads/functional-query/queries/QueryTest/processing-cost-admission-slots.test
M testdata/workloads/functional-query/queries/QueryTest/runtime_filters.test
M 
testdata/workloads/functional-query/queries/QueryTest/runtime_filters_mt_dop.test
M testdata/workloads/functional-query/queries/QueryTest/runtime_row_filters.test
M 
testdata/workloads/functional-query/queries/QueryTest/union-const-scalar-expr-codegen.test
M testdata/workloads/tpcds-insert/queries/partitioned-insert.test
M testdata/workloads/tpch/queries/runtime-profile-aggregated.test
M testdata/workloads/tpch/queries/tpch-passthrough-aggregations.test
M testdata/workloads/tpch/queries/tpch-q1.test
M testdata/workloads/tpch/queries/tpch-q10.test
M testdata/workloads/tpch/queries/tpch-q11.test
M testdata/workloads/tpch/queries/tpch-q12.test
M testdata/workloads/tpch/queries/tpch-q13.test
M testdata/workloads/tpch/queries/tpch-q14.test
M testdata/workloads/tpch/queries/tpch-q15.test
M testdata/workloads/tpch/queries/tpch-q16.test
M testdata/workloads/tpch/queries/tpch-q17.test
M testdata/workloads/tpch/queries/tpch-q18.test
M testdata/workloads/tpch/queries/tpch-q19.test
M testdata/workloads/tpch/queries/tpch-q2.test
M testdata/workloads/tpch/queries/tpch-q20.test
M testdata/workloads/tpch/queries/tpch-q21.test
M testdata/workloads/tpch/queries/tpch-q22.test
M testdata/workloads/tpch/queries/tpch-q3.test
M testdata/workloads/tpch/queries/tpch-q4.test
M testdata/workloads/tpch/queries/tpch-q5.test
M testdata/workloads/tpch/queries/tpch-q6.test
M testdata/workloads/tpch/queries/tpch-q7.test
M testdata/workloads/tpch/queries/tpch-q8.test
M testdata/workloads/tpch/queries/tpch-q9.test
M tests/common/impala_test_suite.py
M tests/common/test_result_verifier.py
M tests/custom_cluster/test_admission_controller.py
M tests/custom_cluster/test_executor_groups.py
M tests/custom_cluster/test_mt_dop.py
M tests/custom_cluster/test_query_live.py
M tests/custom_cluster/test_query_log.py
M tests/custom_cluster/test_query_retries.py
M tests/custom_cluster/test_runtime_profile.py
M tests/custom_cluster/test_tuple_cache.py
M tests/query_test/test_fetch.py
M tests/query_test/test_hash_join_timer.py
M tests/query_test/test_iceberg.py
M tests/query_test/test_observability.py
M tests/query_test/test_parquet_bloom_filter.py
M tests/query_test/test_queries.py
M tests/query_test/test_result_spooling.py
M tests/query_test/test_runtime_filters.py
M tests/query_test/test_scanners.py
M tests/query_test/test_sort.py
M tests/unittests/test_result_verifier.py
M tests/util/workload_management.py
87 files changed, 3,619 insertions(+), 3,287 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/54/23154/10
--
To view, visit http://gerrit.cloudera.org:8080/23154
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: If41d6322361fba82c946efd614cc7d28cb1c36e8
Gerrit-Change-Number: 23154
Gerrit-PatchSet: 10
Gerrit-Owner: Surya Hebbar <sheb...@cloudera.com>
Gerrit-Reviewer: Abhishek Rawat <ara...@cloudera.com>
Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Kurt Deschler <kdesc...@cloudera.com>
Gerrit-Reviewer: Riza Suminto <riza.sumi...@cloudera.com>

Reply via email to