Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/20798 )

Change subject: IMPALA-12632: Use Atomics for CpuUsageRatio counters
......................................................................


Patch Set 1:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/20798/1/be/src/util/system-state-info.cc
File be/src/util/system-state-info.cc:

http://gerrit.cloudera.org:8080/#/c/20798/1/be/src/util/system-state-info.cc@150
PS1, Line 150:   cpu_ratios_.system.Store(((cur[CPU_SYSTEM] - old[CPU_SYSTEM]) 
* BASIS_MAX) / total_tics);
line too long (91 > 90)


http://gerrit.cloudera.org:8080/#/c/20798/1/be/src/util/system-state-info.cc@151
PS1, Line 151:   cpu_ratios_.iowait.Store(((cur[CPU_IOWAIT] - old[CPU_IOWAIT]) 
* BASIS_MAX) / total_tics);
line too long (91 > 90)



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I373cb8ae1a45e5ec07318ccb5870e65efc906cca
Gerrit-Change-Number: 20798
Gerrit-PatchSet: 1
Gerrit-Owner: Andrew Sherman <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Comment-Date: Fri, 15 Dec 2023 00:57:39 +0000
Gerrit-HasComments: Yes

Reply via email to