Repository: ambari Updated Branches: refs/heads/branch-2.5 2c3caa776 -> d21f1f6e7
AMBARI-18185 : Selecting one host when topN is set, throws an error. (avijayan) Project: http://git-wip-us.apache.org/repos/asf/ambari/repo Commit: http://git-wip-us.apache.org/repos/asf/ambari/commit/cd778862 Tree: http://git-wip-us.apache.org/repos/asf/ambari/tree/cd778862 Diff: http://git-wip-us.apache.org/repos/asf/ambari/diff/cd778862 Branch: refs/heads/branch-2.5 Commit: cd7788621d22750156e8df52a6ac94faba49d07a Parents: 2c3caa7 Author: Aravindan Vijayan <[email protected]> Authored: Tue Sep 13 20:16:03 2016 -0700 Committer: Aravindan Vijayan <[email protected]> Committed: Wed Sep 14 15:05:18 2016 -0700 ---------------------------------------------------------------------- .../metrics/timeline/HBaseTimelineMetricStore.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/ambari/blob/cd778862/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 0b82506..686533b 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 @@ -201,7 +201,8 @@ public class HBaseTimelineMetricStore extends AbstractService implements Timelin .grouped(groupedByHosts); if (topNConfig != null) { - if (TopNCondition.isTopNHostCondition(metricNames, hostnames) || TopNCondition.isTopNMetricCondition(metricNames, hostnames)) { + if (TopNCondition.isTopNHostCondition(metricNames, hostnames) ^ //Only 1 condition should be true. + TopNCondition.isTopNMetricCondition(metricNames, hostnames)) { conditionBuilder.topN(topNConfig.getTopN()); conditionBuilder.isBottomN(topNConfig.getIsBottomN()); Function.ReadFunction readFunction = Function.ReadFunction.getFunction(topNConfig.getTopNFunction());
