Browse Source

Forbid negative values for index.unassigned.node_left.delayed_timeout (#26828)

Change delayed_timeout to be a positiveTimeSetting, and add note that this is a breaking change
David Turner 8 years ago
parent
commit
8fe9a20982

+ 1 - 1
core/src/main/java/org/elasticsearch/cluster/routing/UnassignedInfo.java

@@ -50,7 +50,7 @@ public final class UnassignedInfo implements ToXContentFragment, Writeable {
     public static final FormatDateTimeFormatter DATE_TIME_FORMATTER = Joda.forPattern("dateOptionalTime");
     public static final FormatDateTimeFormatter DATE_TIME_FORMATTER = Joda.forPattern("dateOptionalTime");
 
 
     public static final Setting<TimeValue> INDEX_DELAYED_NODE_LEFT_TIMEOUT_SETTING =
     public static final Setting<TimeValue> INDEX_DELAYED_NODE_LEFT_TIMEOUT_SETTING =
-        Setting.timeSetting("index.unassigned.node_left.delayed_timeout", TimeValue.timeValueMinutes(1), Property.Dynamic,
+        Setting.positiveTimeSetting("index.unassigned.node_left.delayed_timeout", TimeValue.timeValueMinutes(1), Property.Dynamic,
             Property.IndexScope);
             Property.IndexScope);
     /**
     /**
      * Reason why the shard is in unassigned state.
      * Reason why the shard is in unassigned state.

+ 5 - 0
docs/reference/migration/migrate_7_0/indices.asciidoc

@@ -5,3 +5,8 @@
 
 
 Due to cross-cluster search using `:` to separate a cluster and index name,
 Due to cross-cluster search using `:` to separate a cluster and index name,
 index names may no longer contain `:`.
 index names may no longer contain `:`.
+
+==== `index.unassigned.node_left.delayed_timeout` may no longer be negative
+
+Negative values were interpreted as zero in earlier versions but are no
+longer accepted.