Prathmesh Sapate has uploaded a new patch set (#2). ( http://gerrit.cloudera.org:8080/24539 )
Change subject: IMPALA-12391: Include event sequence names in runtime profile JSON ...................................................................... IMPALA-12391: Include event sequence names in runtime profile JSON When retrieving profiles in JSON format, event_sequences was represented as an array of sequences instead of a map. The EventSequence names (e.g. "Query Compilation", "Query Timeline") were missing. Change event_sequences_json to use kObjectType and add the EventSequence names as object keys in RuntimeProfile and AggregatedRuntimeProfile ToJson. Testing: - Updated ToJson.RuntimeProfileToJsonTest for object-keyed sequences. - Updated AggregatedEventSequenceToJsonTest. - Regenerate expected JSON with impala-profile-tool after event_sequences changed from array to object keyed by sequence name. Fixes TestProfileTool.test_json_output and test_json_output_profile_v2 CI failures. Change-Id: I72d3f4bef00fd5a809878724e30d1275e5989903 --- M be/src/util/runtime-profile-test.cc M be/src/util/runtime-profile.cc 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_extended.expected.pretty.json M testdata/impala-profiles/impala_profile_log_tpcds_compute_stats_v2.expected.json M testdata/impala-profiles/impala_profile_log_tpcds_compute_stats_v2_extended.expected.pretty.json 7 files changed, 1,211 insertions(+), 613 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/39/24539/2 -- To view, visit http://gerrit.cloudera.org:8080/24539 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I72d3f4bef00fd5a809878724e30d1275e5989903 Gerrit-Change-Number: 24539 Gerrit-PatchSet: 2 Gerrit-Owner: Prathmesh Sapate <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
