summaryrefslogtreecommitdiffstats
path: root/container-core
diff options
context:
space:
mode:
authorHenning Baldersheim <balder@yahoo-inc.com>2017-02-05 18:32:44 +0100
committerHenning Baldersheim <balder@yahoo-inc.com>2017-02-06 04:26:31 +0200
commitfb6fdbac7605d1891c56536a6873f33870f8e972 (patch)
treefc8c6b3a1404435539ccf5666fad1982dbc67c98 /container-core
parent5355427111df0ef2700b594022a8054a6f034d6f (diff)
Finalize degraded coverage info and test it's json rendering.
Diffstat (limited to 'container-core')
-rw-r--r--container-core/src/main/java/com/yahoo/container/handler/Coverage.java7
1 files changed, 6 insertions, 1 deletions
diff --git a/container-core/src/main/java/com/yahoo/container/handler/Coverage.java b/container-core/src/main/java/com/yahoo/container/handler/Coverage.java
index 6c1cfb0fab3..8ff56595f1a 100644
--- a/container-core/src/main/java/com/yahoo/container/handler/Coverage.java
+++ b/container-core/src/main/java/com/yahoo/container/handler/Coverage.java
@@ -2,6 +2,8 @@
package com.yahoo.container.handler;
+import com.google.common.annotations.Beta;
+
/**
* The coverage report for a result set.
*
@@ -105,15 +107,18 @@ public class Coverage {
/**
* Total number of documents that will be searchable once redistribution has settled.
+ * Still in beta, sematics not finalized yet.
*
* @return Total number of documents that will soon be available.
*/
+ @Beta
public long getSoonActive() { return soonActive; }
- public boolean isDegraded() { return degradedReason != 0; }
+ public boolean isDegraded() { return (degradedReason != 0) || isDegradedByNonIdealState(); }
public boolean isDegradedByMatchPhase() { return (degradedReason & DEGRADED_BY_MATCH_PHASE) != 0; }
public boolean isDegradedByTimeout() { return (degradedReason & DEGRADED_BY_TIMEOUT) != 0; }
public boolean isDegradedByAdapativeTimeout() { return (degradedReason & DEGRADED_BY_ADAPTIVE_TIMEOUT) != 0; }
+ public boolean isDegradedByNonIdealState() { return (degradedReason == 0) && (getResultPercentage() != 100);}
/**
* @return whether the search had full coverage or not