ambari-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From swa...@apache.org
Subject ambari git commit: AMBARI-9838. Ambari Metrics broken for Slider deployed apps - application id was made mandatory. (swagle)
Date Fri, 27 Feb 2015 21:31:13 GMT
Repository: ambari
Updated Branches:
  refs/heads/trunk 57b8c7190 -> 5bc11b68b


AMBARI-9838. Ambari Metrics broken for Slider deployed apps - application id was made mandatory.
(swagle)


Project: http://git-wip-us.apache.org/repos/asf/ambari/repo
Commit: http://git-wip-us.apache.org/repos/asf/ambari/commit/5bc11b68
Tree: http://git-wip-us.apache.org/repos/asf/ambari/tree/5bc11b68
Diff: http://git-wip-us.apache.org/repos/asf/ambari/diff/5bc11b68

Branch: refs/heads/trunk
Commit: 5bc11b68b8be47ed21da0ce44e14a0645eb41ff0
Parents: 57b8c71
Author: Siddharth Wagle <swagle@hortonworks.com>
Authored: Fri Feb 27 13:31:04 2015 -0800
Committer: Siddharth Wagle <swagle@hortonworks.com>
Committed: Fri Feb 27 13:31:04 2015 -0800

----------------------------------------------------------------------
 .../metrics/timeline/HBaseTimelineMetricStore.java            | 7 -------
 1 file changed, 7 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ambari/blob/5bc11b68/ambari-metrics/ambari-metrics-timelineservice/src/main/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/HBaseTimelineMetricStore.java
----------------------------------------------------------------------
diff --git a/ambari-metrics/ambari-metrics-timelineservice/src/main/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/HBaseTimelineMetricStore.java
b/ambari-metrics/ambari-metrics-timelineservice/src/main/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/HBaseTimelineMetricStore.java
index 3969fa9..a4980b4 100644
--- a/ambari-metrics/ambari-metrics-timelineservice/src/main/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/HBaseTimelineMetricStore.java
+++ b/ambari-metrics/ambari-metrics-timelineservice/src/main/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/HBaseTimelineMetricStore.java
@@ -109,13 +109,9 @@ public class HBaseTimelineMetricStore extends AbstractService
       Long startTime, Long endTime, Precision precision, Integer limit,
       boolean groupedByHosts) throws SQLException, IOException {
 
-
     if (metricNames == null || metricNames.isEmpty()) {
       throw new IllegalArgumentException("No metric name filter specified.");
     }
-    if (applicationId == null || applicationId.trim().length() == 0) {
-      throw new IllegalArgumentException("No applicationID filter specified.");
-    }
     if ((startTime == null && endTime != null)
         || (startTime != null && endTime == null)) {
       throw new IllegalArgumentException("Open ended query not supported ");
@@ -222,9 +218,6 @@ public class HBaseTimelineMetricStore extends AbstractService
     if (metricName == null || metricName.isEmpty()) {
       throw new IllegalArgumentException("No metric name filter specified.");
     }
-    if (applicationId == null || applicationId.trim().length() == 0) {
-      throw new IllegalArgumentException("No applicationID filter specified.");
-    }
     if ((startTime == null && endTime != null)
         || (startTime != null && endTime == null)) {
       throw new IllegalArgumentException("Open ended query not supported ");


Mime
View raw message