Repository: tajo Updated Branches: refs/heads/master bdd102c8f -> 008940ccb
TAJO-1689: Metrics file reporter prints histogram metric without group name. Closes #633 Project: http://git-wip-us.apache.org/repos/asf/tajo/repo Commit: http://git-wip-us.apache.org/repos/asf/tajo/commit/008940cc Tree: http://git-wip-us.apache.org/repos/asf/tajo/tree/008940cc Diff: http://git-wip-us.apache.org/repos/asf/tajo/diff/008940cc Branch: refs/heads/master Commit: 008940ccbd8eb85365c37767581e427d4ea6400c Parents: bdd102c Author: Hyunsik Choi <[email protected]> Authored: Mon Jul 20 20:49:45 2015 +0900 Committer: Hyunsik Choi <[email protected]> Committed: Mon Jul 20 20:52:49 2015 +0900 ---------------------------------------------------------------------- CHANGES | 3 +++ .../apache/tajo/util/metrics/reporter/TajoMetricsReporter.java | 6 +++++- 2 files changed, 8 insertions(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/tajo/blob/008940cc/CHANGES ---------------------------------------------------------------------- diff --git a/CHANGES b/CHANGES index 6804e73..0ec1699 100644 --- a/CHANGES +++ b/CHANGES @@ -177,6 +177,9 @@ Release 0.11.0 - unreleased BUG FIXES + TAJO-1689: Metrics file reporter prints histogram metric without group name. + (hyunsik) + TAJO-1657: Tajo Rest API /database/{database-name]/tables should return table names only without invalid external table info. (Contributed by DaeMyung Kang, Committed by jihoon) http://git-wip-us.apache.org/repos/asf/tajo/blob/008940cc/tajo-core/src/main/java/org/apache/tajo/util/metrics/reporter/TajoMetricsReporter.java ---------------------------------------------------------------------- diff --git a/tajo-core/src/main/java/org/apache/tajo/util/metrics/reporter/TajoMetricsReporter.java b/tajo-core/src/main/java/org/apache/tajo/util/metrics/reporter/TajoMetricsReporter.java index a7e0559..584fcbe 100644 --- a/tajo-core/src/main/java/org/apache/tajo/util/metrics/reporter/TajoMetricsReporter.java +++ b/tajo-core/src/main/java/org/apache/tajo/util/metrics/reporter/TajoMetricsReporter.java @@ -160,7 +160,11 @@ public abstract class TajoMetricsReporter { if(hostAndPort != null && !hostAndPort.isEmpty()) { sb.append(hostAndPort).append(" "); } - sb.append("histo").append(" "); + sb.append("histogram").append(" "); + + if(!groupName.isEmpty()) { + sb.append(groupName).append(" "); + } String prefix = ""; for(Map.Entry<String, Histogram> eachHistogram: histograms.entrySet()) {
