From b6e5ccaf8a2304892405712cf832d23efdce0940 Mon Sep 17 00:00:00 2001 From: Przemyslaw Gomulka Date: Thu, 17 Jan 2019 09:10:09 +0100 Subject: [PATCH] Remove the AbstracLifecycleComponent constructor with Settings (#37523) Adding the migration guide and removing the deprecated in 6.x constructor relates #35560 relates #34488 --- docs/reference/migration/migrate_7_0/java.asciidoc | 8 +++++++- .../common/component/AbstractLifecycleComponent.java | 6 ------ 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/docs/reference/migration/migrate_7_0/java.asciidoc b/docs/reference/migration/migrate_7_0/java.asciidoc index e48a4cf1b45..f34b1c6ca99 100644 --- a/docs/reference/migration/migrate_7_0/java.asciidoc +++ b/docs/reference/migration/migrate_7_0/java.asciidoc @@ -38,4 +38,10 @@ because `Settings` is no longer needed. ==== Deprecated method `Client#termVector` removed The client method `termVector`, deprecated in 2.0, has been removed. The method -`termVectors` (plural) should be used instead. \ No newline at end of file +`termVectors` (plural) should be used instead. + +[float] +==== Deprecated constructor `AbstractLifecycleComponent(Settings settings)` removed + +The constructor `AbstractLifecycleComponent(Settings settings)`, deprecated in 6.7 +has been removed. The parameterless constructor should be used instead. diff --git a/server/src/main/java/org/elasticsearch/common/component/AbstractLifecycleComponent.java b/server/src/main/java/org/elasticsearch/common/component/AbstractLifecycleComponent.java index 2caaa43fbcd..772d2d89cf5 100644 --- a/server/src/main/java/org/elasticsearch/common/component/AbstractLifecycleComponent.java +++ b/server/src/main/java/org/elasticsearch/common/component/AbstractLifecycleComponent.java @@ -21,7 +21,6 @@ package org.elasticsearch.common.component; import org.apache.logging.log4j.LogManager; import org.apache.logging.log4j.Logger; -import org.elasticsearch.common.settings.Settings; import java.io.IOException; import java.util.List; @@ -36,11 +35,6 @@ public abstract class AbstractLifecycleComponent implements LifecycleComponent { protected AbstractLifecycleComponent() {} - @Deprecated - protected AbstractLifecycleComponent(Settings settings) { - // TODO drop settings from ctor - } - @Override public Lifecycle.State lifecycleState() { return this.lifecycle.state();