hive-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jimmy Xiang (JIRA)" <>
Subject [jira] [Commented] (HIVE-12987) Add metrics for HS2 active users and SQL operations
Date Wed, 03 Feb 2016 22:39:40 GMT


Jimmy Xiang commented on HIVE-12987:

Sure. Thanks a lot for reviewing it.

bq. 1. The overriding of the metrics-prefix strings seems confusing, can't we just add some
new constants/prefixes in MetricsConstants and have Sqloperation increment/decrement those
explicitly as part of overriding the setMetrics method?
And moreover with this code, I think we may be not getting some old metrics in case of SQLOperation,
wouldn't it stop increment/decrement the total count?

The idea is to separate to the SQL operations from other operations.  We have two choices.
The first one is the same as we said, we don't touch the existing metris, just add a new one
for SQL operation. The second one is what I implemented. The original operation metrics is
changed so that it doesn't include the SQL operations any more.

bq. 2. The increment/decrementUserQuery methods also seem a bit complicated. Why do we need
extra map in SQLOperation? I think the counter-map already in CodahaleMetrics should take
care of all the stuff the new map does?
I'd like to change the user count only when a SQL query is executed. Can I do this with the
map in CodahaleMetrics?

> Add metrics for HS2 active users and SQL operations
> ---------------------------------------------------
>                 Key: HIVE-12987
>                 URL:
>             Project: Hive
>          Issue Type: Task
>            Reporter: Jimmy Xiang
>            Assignee: Jimmy Xiang
>         Attachments: HIVE-12987.1.patch, HIVE-12987.2.patch
> HIVE-12271 added metrics for all HS2 operations. Sometimes, users are also interested
in metrics just for SQL operations.
> It is useful to track active user count as well.

This message was sent by Atlassian JIRA

View raw message