summaryrefslogtreecommitdiffstats
path: root/clustercontroller-core
diff options
context:
space:
mode:
authorGeir Storli <geirst@oath.com>2018-03-07 15:45:29 +0100
committerGeir Storli <geirst@oath.com>2018-03-07 15:45:29 +0100
commit47074041366eb553997843c66849788beb127045 (patch)
tree9898c3b8303592316f22e2a07710540053ac2eff /clustercontroller-core
parent857377752b721ebf2cca578637277bb518e3e261 (diff)
Simplifications.
Diffstat (limited to 'clustercontroller-core')
-rw-r--r--clustercontroller-core/src/main/java/com/yahoo/vespa/clustercontroller/core/AggregatedClusterStats.java4
-rw-r--r--clustercontroller-core/src/main/java/com/yahoo/vespa/clustercontroller/core/AggregatedStatsMergePendingChecker.java6
2 files changed, 3 insertions, 7 deletions
diff --git a/clustercontroller-core/src/main/java/com/yahoo/vespa/clustercontroller/core/AggregatedClusterStats.java b/clustercontroller-core/src/main/java/com/yahoo/vespa/clustercontroller/core/AggregatedClusterStats.java
index 7a00627f7ae..ce2cd19b578 100644
--- a/clustercontroller-core/src/main/java/com/yahoo/vespa/clustercontroller/core/AggregatedClusterStats.java
+++ b/clustercontroller-core/src/main/java/com/yahoo/vespa/clustercontroller/core/AggregatedClusterStats.java
@@ -6,8 +6,8 @@ package com.yahoo.vespa.clustercontroller.core;
*/
public interface AggregatedClusterStats {
- public boolean hasUpdatesFromAllDistributors();
+ boolean hasUpdatesFromAllDistributors();
- public ContentClusterStats getStats();
+ ContentClusterStats getStats();
}
diff --git a/clustercontroller-core/src/main/java/com/yahoo/vespa/clustercontroller/core/AggregatedStatsMergePendingChecker.java b/clustercontroller-core/src/main/java/com/yahoo/vespa/clustercontroller/core/AggregatedStatsMergePendingChecker.java
index 71a582a2a4b..88b7d0d877b 100644
--- a/clustercontroller-core/src/main/java/com/yahoo/vespa/clustercontroller/core/AggregatedStatsMergePendingChecker.java
+++ b/clustercontroller-core/src/main/java/com/yahoo/vespa/clustercontroller/core/AggregatedStatsMergePendingChecker.java
@@ -26,11 +26,7 @@ public class AggregatedStatsMergePendingChecker implements MergePendingChecker {
ContentNodeStats nodeStats = stats.getStats().getContentNode(contentNodeIndex);
if (nodeStats != null) {
ContentNodeStats.BucketSpaceStats bucketSpaceStats = nodeStats.getBucketSpace(bucketSpace);
- if (bucketSpaceStats != null && bucketSpaceStats.mayHaveBucketsPending()) {
- return true;
- } else {
- return false;
- }
+ return (bucketSpaceStats != null && bucketSpaceStats.mayHaveBucketsPending());
}
return true;
}