hive-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hive QA (JIRA)" <>
Subject [jira] [Commented] (HIVE-16206) Make Codahale metrics reporters pluggable
Date Mon, 20 Mar 2017 18:43:41 GMT


Hive QA commented on HIVE-16206:

Here are the results of testing the latest attachment:

{color:green}SUCCESS:{color} +1 due to 1 test(s) being added or modified.

{color:red}ERROR:{color} -1 due to 1 failed/errored test(s), 10478 tests executed
*Failed tests:*
org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[comments] (batchId=35)

Test results:
Console output:
Test logs:

Executing org.apache.hive.ptest.execution.TestCheckPhase
Executing org.apache.hive.ptest.execution.PrepPhase
Executing org.apache.hive.ptest.execution.ExecutionPhase
Executing org.apache.hive.ptest.execution.ReportingPhase
Tests exited with: TestsFailedException: 1 tests failed

This message is automatically generated.

ATTACHMENT ID: 12859579 - PreCommit-HIVE-Build

> Make Codahale metrics reporters pluggable
> -----------------------------------------
>                 Key: HIVE-16206
>                 URL:
>             Project: Hive
>          Issue Type: Improvement
>          Components: Metastore
>    Affects Versions: 2.1.2
>            Reporter: Sunitha Beeram
>            Assignee: Sunitha Beeram
>         Attachments: HIVE-16206.2.patch, HIVE-16206.patch
> Hive metrics code currently allows pluggable metrics handlers - ie, handlers that take
care of providing interfaces for metrics collection as well as a reporting; one of the 'handlers'
is CodahaleMetrics. Codahale can work with different reporters - currently supported ones
are Console, JMX, JSON file and hadoop2 sink. However, adding a new reporter involves changing
that class. We would like to make this conf driven just the way MetricsFactory handles configurable
Metrics classes.
> Scope of work:
> - Provide a new configuration option, HIVE_CODAHALE_REPORTER_CLASSES that enumerates
> - Move JsonFileReporter into its own class.
> - Update to read new config option and if the new option is not
present, look for the old option and instantiate accordingly) - ie, make the code backward
> - Update and add new tests.

This message was sent by Atlassian JIRA

View raw message