Csaba Ringhofer has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/20473 )

Change subject: IMPALA-11553: Add event specific metrics in the table metrics
......................................................................


Patch Set 8: Code-Review+1

(3 comments)

http://gerrit.cloudera.org:8080/#/c/20473/5//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/20473/5//COMMIT_MSG@11
PS5, Line 11:  identif
> I have attached the screenshots of the following table metrics: events-proc
I see, so this is simply how Impala treats gauges and not something specific to 
this metric, right?


http://gerrit.cloudera.org:8080/#/c/20473/8//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/20473/8//COMMIT_MSG@11
PS8, Line 11: mertic
typo


http://gerrit.cloudera.org:8080/#/c/20473/5/fe/src/main/java/org/apache/impala/catalog/events/MetastoreEvents.java
File fe/src/main/java/org/apache/impala/catalog/events/MetastoreEvents.java:

http://gerrit.cloudera.org:8080/#/c/20473/5/fe/src/main/java/org/apache/impala/catalog/events/MetastoreEvents.java@1745
PS5, Line 1745: ventProcessing
> For rename operation, we accumulate and associate the process time with the
ack



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I2428029361e610a0fcd8ed11be2ab771f03b00dd
Gerrit-Change-Number: 20473
Gerrit-PatchSet: 8
Gerrit-Owner: Sai Hemanth Gantasala <saihema...@cloudera.com>
Gerrit-Reviewer: Anonymous Coward <k.venureddy2...@gmail.com>
Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Quanlong Huang <huangquanl...@gmail.com>
Gerrit-Reviewer: Sai Hemanth Gantasala <saihema...@cloudera.com>
Gerrit-Comment-Date: Wed, 20 Dec 2023 10:02:31 +0000
Gerrit-HasComments: Yes

Reply via email to