From 9ccf617a9576d152933ecdddab3c6bc6e77725f7 Mon Sep 17 00:00:00 2001
From: Dimitrios Athanasiou <dimitris@elastic.co>
Date: Thu, 11 May 2017 16:26:00 +0100
Subject: [PATCH] [TEST] Add explanation for custom random time value in
 ChunkingConfigTests

Original commit: elastic/x-pack-elasticsearch@9e03fb833f617abbd0bb97d10c7d1eeb9013da6a
---
 .../org/elasticsearch/xpack/ml/datafeed/ChunkingConfigTests.java | 1 +
 1 file changed, 1 insertion(+)

diff --git a/plugin/src/test/java/org/elasticsearch/xpack/ml/datafeed/ChunkingConfigTests.java b/plugin/src/test/java/org/elasticsearch/xpack/ml/datafeed/ChunkingConfigTests.java
index 505ee27ad7f..9b06ac8de2a 100644
--- a/plugin/src/test/java/org/elasticsearch/xpack/ml/datafeed/ChunkingConfigTests.java
+++ b/plugin/src/test/java/org/elasticsearch/xpack/ml/datafeed/ChunkingConfigTests.java
@@ -54,6 +54,7 @@ public class ChunkingConfigTests extends AbstractSerializingTestCase<ChunkingCon
         ChunkingConfig.Mode mode = randomFrom(ChunkingConfig.Mode.values());
         TimeValue timeSpan = null;
         if (mode == ChunkingConfig.Mode.MANUAL) {
+            // time span is required to be at least 1 millis, so we use a custom method to generate a time value here
             timeSpan = randomPositiveSecondsMinutesHours();
         }
         return new ChunkingConfig(mode, timeSpan);