Surya Hebbar has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/20008 )

Change subject: IMPALA-12182: Add CPU utilization chart for RuntimeProfile's 
sampled metrics
......................................................................


Patch Set 1:

(3 comments)

http://gerrit.cloudera.org:8080/#/c/20008/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/20008/1//COMMIT_MSG@15
PS1, Line 15: AggregatedRuntimeProfile
> This should be RuntimeProfile instead of AggregatedRuntimeProfile. Aggregat
Sorry, I was referring to the thrift profile mentioned on the query profile 
page.

(Showing 52 of 209 values from Thrift Profile)

To provide some insight into, where all utilization values were present.

I have removed this from the commit message now.


http://gerrit.cloudera.org:8080/#/c/20008/1/.gitattributes
File .gitattributes:

http://gerrit.cloudera.org:8080/#/c/20008/1/.gitattributes@31
PS1, Line 31: www/chartist/chartist.umd.min.js binary
            : www/chartist/chartist.min.css binary
> Insert these two lines in the right position to keep alphabet order
Done


http://gerrit.cloudera.org:8080/#/c/20008/1/LICENSE.txt
File LICENSE.txt:

http://gerrit.cloudera.org:8080/#/c/20008/1/LICENSE.txt@1123
PS1, Line 1123: Permission is hereby granted, free of charge, to any person 
obtaining a copy of this software and a
> Okay.
Done



--
To view, visit http://gerrit.cloudera.org:8080/20008
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Idea2a6db217dbfaa7a0695aeabb6d9c1ecf62158
Gerrit-Change-Number: 20008
Gerrit-PatchSet: 1
Gerrit-Owner: Surya Hebbar <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Kurt Deschler <[email protected]>
Gerrit-Reviewer: Riza Suminto <[email protected]>
Gerrit-Reviewer: Surya Hebbar <[email protected]>
Gerrit-Reviewer: Wenzhe Zhou <[email protected]>
Gerrit-Comment-Date: Tue, 20 Jun 2023 09:34:00 +0000
Gerrit-HasComments: Yes

Reply via email to