[ https://issues.apache.org/jira/browse/HIVE-22140?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Laszlo Bodor updated HIVE-22140: -------------------------------- Description: Currently, different default value and even different unit are used for json metric file generation frequency (for metastore and hiveserver2), which is confusing, especially in a situation when a metastore config can fall back to a corresponding hive config. I think ms would be appropriate for both of them, without even changing the default 1m for metastore (so changing it to 60000ms) hive.service.metrics.file.frequency Hiveserver2: https://github.com/apache/hive/blob/master/common/src/java/org/apache/hadoop/hive/conf/HiveConf.java {code:java} HIVE_METRICS_JSON_FILE_INTERVAL("hive.service.metrics.file.frequency", "5000ms", new TimeValidator(TimeUnit.MILLISECONDS), "For metric class org.apache.hadoop.hive.common.metrics.metrics2.JsonFileMetricsReporter, " + "the frequency of updating JSON metrics file."), {code} Metastore: https://github.com/apache/hive/blob/master/standalone-metastore/metastore-common/src/main/java/org/apache/hadoop/hive/metastore/conf/MetastoreConf.java {code:java} METRICS_JSON_FILE_INTERVAL("metastore.metrics.file.frequency", "hive.service.metrics.file.frequency", 1, TimeUnit.MINUTES, "For json metric reporter, the frequency of updating JSON metrics file."), {code} was: Currently, different default value and even different unit are used for json metric file generation frequency, which is confusing, especially in a situation when a metastore config can fall back to hive a corresponding hive config. I think ms would be appropriate for both of them, without even changing the default 1m for metastore (so changing it to 60000ms) Hiveserver2: https://github.com/apache/hive/blob/master/common/src/java/org/apache/hadoop/hive/conf/HiveConf.java {code:java} HIVE_METRICS_JSON_FILE_INTERVAL("hive.service.metrics.file.frequency", "5000ms", new TimeValidator(TimeUnit.MILLISECONDS), "For metric class org.apache.hadoop.hive.common.metrics.metrics2.JsonFileMetricsReporter, " + "the frequency of updating JSON metrics file."), {code} Metastore: https://github.com/apache/hive/blob/master/standalone-metastore/metastore-common/src/main/java/org/apache/hadoop/hive/metastore/conf/MetastoreConf.java {code:java} METRICS_JSON_FILE_INTERVAL("metastore.metrics.file.frequency", "hive.service.metrics.file.frequency", 1, TimeUnit.MINUTES, "For json metric reporter, the frequency of updating JSON metrics file."), {code} > Metrics: unify codehale metric frequency unit and default between metastore > and hiveserver2 > ------------------------------------------------------------------------------------------- > > Key: HIVE-22140 > URL: https://issues.apache.org/jira/browse/HIVE-22140 > Project: Hive > Issue Type: Bug > Components: Metastore > Reporter: Laszlo Bodor > Priority: Major > > Currently, different default value and even different unit are used for json > metric file generation frequency (for metastore and hiveserver2), which is > confusing, especially in a situation when a metastore config can fall back to > a corresponding hive config. > I think ms would be appropriate for both of them, without even changing the > default 1m for metastore (so changing it to 60000ms) > hive.service.metrics.file.frequency > Hiveserver2: > https://github.com/apache/hive/blob/master/common/src/java/org/apache/hadoop/hive/conf/HiveConf.java > {code:java} > HIVE_METRICS_JSON_FILE_INTERVAL("hive.service.metrics.file.frequency", > "5000ms", > new TimeValidator(TimeUnit.MILLISECONDS), > "For metric class > org.apache.hadoop.hive.common.metrics.metrics2.JsonFileMetricsReporter, " + > "the frequency of updating JSON metrics file."), > {code} > Metastore: > https://github.com/apache/hive/blob/master/standalone-metastore/metastore-common/src/main/java/org/apache/hadoop/hive/metastore/conf/MetastoreConf.java > {code:java} > METRICS_JSON_FILE_INTERVAL("metastore.metrics.file.frequency", > "hive.service.metrics.file.frequency", 1, TimeUnit.MINUTES, > "For json metric reporter, the frequency of updating JSON metrics > file."), > {code} -- This message was sent by Atlassian Jira (v8.3.2#803003)