You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@impala.apache.org by ta...@apache.org on 2016/12/15 23:00:50 UTC

[24/50] [abbrv] incubator-impala git commit: IMPALA-4609: prefix thread counters in fragment profile

IMPALA-4609: prefix thread counters in fragment profile

This adds a prefix to the counter names so that it's more obvious that
the thread counters are aggregates across all fragment threads.

Example output for "select * from tpch_parquet.lineitem where l_orderkey < 0":

 Averaged Fragment F00:(Total: 102.949ms, non-child: 0.000ns, % non-child: 0.00%)
      split sizes:  min: 55.70 MB, max: 69.10 MB, avg: 64.54 MB, stddev: 6.25 MB
      completion times: min:4s629ms  max:5s542ms  mean: 5s197ms  stddev:404.911ms
      execution rates: min:12.03 MB/sec  max:12.69 MB/sec  mean:12.40 MB/sec  stddev:281.88 KB/sec
      num instances: 3
       - AverageThreadTokens: 2.00
       - BloomFilterBytes: 0
       - PeakMemoryUsage: 88.46 MB (92753209)
       - PerHostPeakMemUsage: 88.46 MB (92755940)
       - RowsProduced: 0 (0)
       - TotalNetworkReceiveTime: 0.000ns
       - TotalNetworkSendTime: 307.000ns
       - TotalStorageWaitTime: 414.738ms
       - TotalThreadsInvoluntaryContextSwitches: 62 (62)
       - TotalThreadsTotalWallClockTime: 10s228ms
         - TotalThreadsSysTime: 118.666ms
         - TotalThreadsUserTime: 4s630ms
       - TotalThreadsVoluntaryContextSwitches: 174 (174)

Change-Id: Icb8cfbddc758d06b25a14343310bfd9a932ad1f0
Reviewed-on: http://gerrit.cloudera.org:8080/5392
Reviewed-by: Tim Armstrong <ta...@cloudera.com>
Tested-by: Impala Public Jenkins


Project: http://git-wip-us.apache.org/repos/asf/incubator-impala/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-impala/commit/1762dd1b
Tree: http://git-wip-us.apache.org/repos/asf/incubator-impala/tree/1762dd1b
Diff: http://git-wip-us.apache.org/repos/asf/incubator-impala/diff/1762dd1b

Branch: refs/heads/hadoop-next
Commit: 1762dd1b815b43d4507ce28e1879091dd2e4de57
Parents: 80f8517
Author: Tim Armstrong <ta...@cloudera.com>
Authored: Tue Dec 6 18:27:30 2016 -0800
Committer: Impala Public Jenkins <im...@gerrit.cloudera.org>
Committed: Fri Dec 9 02:19:41 2016 +0000

----------------------------------------------------------------------
 be/src/runtime/runtime-state.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-impala/blob/1762dd1b/be/src/runtime/runtime-state.cc
----------------------------------------------------------------------
diff --git a/be/src/runtime/runtime-state.cc b/be/src/runtime/runtime-state.cc
index 7954f22..a0c3335 100644
--- a/be/src/runtime/runtime-state.cc
+++ b/be/src/runtime/runtime-state.cc
@@ -134,7 +134,7 @@ Status RuntimeState::Init(ExecEnv* exec_env) {
     DCHECK(resource_pool_ != NULL);
   }
 
-  total_thread_statistics_ = ADD_THREAD_COUNTERS(runtime_profile(), "");
+  total_thread_statistics_ = ADD_THREAD_COUNTERS(runtime_profile(), "TotalThreads");
   total_storage_wait_timer_ = ADD_TIMER(runtime_profile(), "TotalStorageWaitTime");
   total_network_send_timer_ = ADD_TIMER(runtime_profile(), "TotalNetworkSendTime");
   total_network_receive_timer_ = ADD_TIMER(runtime_profile(), "TotalNetworkReceiveTime");