anujphadke has posted comments on this change. Change subject: IMPALA-3342: Runtime profile TotalCpuTime should eliminate wait times ......................................................................
Patch Set 1: I think we already print enough information by calling - getrusage(RUSAGE_THREAD, &usage) and obtain the total wall clock time, systime and the user time in the profile The usage struct contains these 2 fields struct timeval ru_utime; /* user CPU time used */ struct timeval ru_stime; /* system CPU time used */ The ru_utime value matches the time reported by using the new timer I have added. So maybe we already print the required information in the profile. We can added the new timer class and we can reuse it later somewhere when we need to measure the CPU time spent. -- To view, visit http://gerrit.cloudera.org:8080/4052 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: If8edd0c3559968ff12e42f83dd970e424fa4a3c8 Gerrit-PatchSet: 1 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: anujphadke <[email protected]> Gerrit-Reviewer: anujphadke <[email protected]> Gerrit-HasComments: No
