[AMBARI-25089] NIFI-Hosts dashboard does not show any metrics in Grafana. (#14) master
authoravijayanhwx <avijayan@hortonworks.com>
Mon, 11 Feb 2019 17:34:11 +0000 (09:34 -0800)
committerGitHub <noreply@github.com>
Mon, 11 Feb 2019 17:34:11 +0000 (09:34 -0800)
ambari-metrics-grafana/ambari-metrics/datasource.js
ambari-metrics-timelineservice/src/main/java/org/apache/ambari/metrics/core/timeline/query/MetadataQueryCondition.java
ambari-metrics-timelineservice/src/test/java/org/apache/ambari/metrics/core/timeline/discovery/TestMetadataManager.java

index 6dc0446..70c3f56 100644 (file)
@@ -291,6 +291,10 @@ define([
         // To speed up querying on templatized dashboards.
         var getAllHostData = function (target) {
           var instanceId = typeof target.templatedCluster == 'undefined' ? '' : '&instanceId=' + target.templatedCluster;
+          var appId = target.app;
+          if ((appId === 'nifi' || appId === 'druid') && !instanceId) {
+            instanceId = "%"
+          }
           var precision = target.precision === 'default' || typeof target.precision == 'undefined' ? '' : '&precision='
             + target.precision;
           var metricAggregator = target.aggregator === "none" ? '' : '._' + target.aggregator;
index f6e8c6a..0f6f611 100644 (file)
@@ -60,7 +60,7 @@ public class MetadataQueryCondition extends TransientMetricCondition {
     }
 
     String instanceId = getInstanceId();
-    if (StringUtils.isNotEmpty(instanceId)) {
+    if (StringUtils.isNotEmpty(instanceId) && !"%".equals(instanceId)) {
       if (instanceId.contains("%")) {
         appendConjunction = append(sb, appendConjunction, instanceId, " INSTANCE_ID LIKE ?");
       } else {
@@ -71,6 +71,11 @@ public class MetadataQueryCondition extends TransientMetricCondition {
     return sb;
   }
 
+  @Override
+  public String getInstanceId() {
+    return instanceId == null || "%".equals(instanceId) || instanceId.isEmpty() ? null : instanceId;
+  }
+
   public boolean isMetricMetadataCondition() {
     return isMetricMetadataCondition;
   }
index 90a5d10..0d9c3e5 100644 (file)
@@ -331,7 +331,7 @@ public class TestMetadataManager extends AbstractMiniHBaseClusterTest {
 
     uuids = metadataManager.getUuidsForGetMetricQuery(Collections.singletonList("%"),
       Collections.singletonList("%"), "%", "%");
-    Assert.assertTrue(uuids.size() == 6);
+    Assert.assertTrue(uuids.size() == 9);
   }
 
   @Test