aboutsummaryrefslogtreecommitdiffstats
path: root/container-search
diff options
context:
space:
mode:
authorTor Brede Vekterli <vekterli@verizonmedia.com>2020-04-22 14:33:57 +0200
committerTor Brede Vekterli <vekterli@verizonmedia.com>2020-04-22 14:33:57 +0200
commit000603b162bf8c28073fde5d56d6a3ee46e48574 (patch)
tree4ffbf225da86caf913363a0b9995f9bbc990fd01 /container-search
parent9ab0ef70e9ed4f422df67603f26bcb0c7918fdc4 (diff)
Increase streaming search timeout trace sampling probabilities
Diffstat (limited to 'container-search')
-rw-r--r--container-search/src/main/java/com/yahoo/vespa/streamingvisitors/TracingOptions.java12
1 files changed, 6 insertions, 6 deletions
diff --git a/container-search/src/main/java/com/yahoo/vespa/streamingvisitors/TracingOptions.java b/container-search/src/main/java/com/yahoo/vespa/streamingvisitors/TracingOptions.java
index 3c96b00d628..b9e19d6a1b6 100644
--- a/container-search/src/main/java/com/yahoo/vespa/streamingvisitors/TracingOptions.java
+++ b/container-search/src/main/java/com/yahoo/vespa/streamingvisitors/TracingOptions.java
@@ -15,12 +15,12 @@ import java.util.concurrent.TimeUnit;
* Encapsulates all trace-related components and options used by the streaming search Searcher.
*
* Provides a DEFAULT static instance which has the following characteristics:
- * - Approximately 1 query every 2 seconds is traced
+ * - Approximately 1 query every second is traced
* - Trace level is set to 7 for traced queries
- * - Only emits traces for queries that have timed out and where the elapsed time is at least 5x
+ * - Only emits traces for queries that have timed out and where the elapsed time is at least 2x
* of the timeout specified in the query itself
* - Emits traces to the Vespa log
- * - Only 1 trace every 10 seconds may be emitted to the log
+ * - Only 2 traces every 10 seconds may be emitted to the log
*/
public class TracingOptions {
@@ -50,11 +50,11 @@ public class TracingOptions {
public static final TracingOptions DEFAULT;
public static final int DEFAULT_TRACE_LEVEL_OVERRIDE = 7; // TODO determine appropriate trace level
// Traces won't be exported unless the query has timed out with a duration that is > timeout * multiplier
- public static final double TRACE_TIMEOUT_MULTIPLIER_THRESHOLD = 5.0;
+ public static final double TRACE_TIMEOUT_MULTIPLIER_THRESHOLD = 2.0;
static {
- SamplingStrategy queryTraceSampler = ProbabilisticSampleRate.withSystemDefaults(0.5);
- SamplingStrategy logExportSampler = MaxSamplesPerPeriod.withSteadyClock(TimeUnit.SECONDS.toNanos(10), 1);
+ SamplingStrategy queryTraceSampler = ProbabilisticSampleRate.withSystemDefaults(1);
+ SamplingStrategy logExportSampler = MaxSamplesPerPeriod.withSteadyClock(TimeUnit.SECONDS.toNanos(10), 2);
TraceExporter traceExporter = new SamplingTraceExporter(new LoggingTraceExporter(), logExportSampler);
DEFAULT = new TracingOptions(queryTraceSampler, traceExporter, System::nanoTime,
DEFAULT_TRACE_LEVEL_OVERRIDE, TRACE_TIMEOUT_MULTIPLIER_THRESHOLD);