honeymoose
/
OpenSearch
mirror of https://github.com/honeymoose/OpenSearch.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

[ML] Remove unused member in NativeAutodetectProcessFactory (elastic/x-pack-elasticsearch#2629) 

Original commit: elastic/x-pack-elasticsearch@31f265f0e5
This commit is contained in:
Dimitris Athanasiou 2017-09-26 14:12:27 +01:00 committed by GitHub
parent 5ad170a7ed
commit b3ae022985
2 changed files with 2 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
plugin/src/main/java/org/elasticsearch/xpack/ml/MachineLearning.java
View File

@ -283,7 +283,7 @@ public class MachineLearning implements ActionPlugin {
// This will only only happen when path.home is not set, which is disallowed in production // This will only only happen when path.home is not set, which is disallowed in production
throw new ElasticsearchException("Failed to create native process controller for Machine Learning"); throw new ElasticsearchException("Failed to create native process controller for Machine Learning");
} }
autodetectProcessFactory = new NativeAutodetectProcessFactory(jobProvider, env, settings, nativeController, internalClient); autodetectProcessFactory = new NativeAutodetectProcessFactory(env, settings, nativeController, internalClient);
normalizerProcessFactory = new NativeNormalizerProcessFactory(env, settings, nativeController); normalizerProcessFactory = new NativeNormalizerProcessFactory(env, settings, nativeController);
} catch (IOException e) { } catch (IOException e) {
// This also should not happen in production, as the MachineLearningFeatureSet should have // This also should not happen in production, as the MachineLearningFeatureSet should have

6
plugin/src/main/java/org/elasticsearch/xpack/ml/job/process/autodetect/NativeAutodetectProcessFactory.java
View File

@ -14,7 +14,6 @@ import org.elasticsearch.common.util.concurrent.EsRejectedExecutionException;
import org.elasticsearch.env.Environment; import org.elasticsearch.env.Environment;
import org.elasticsearch.xpack.ml.job.config.Job; import org.elasticsearch.xpack.ml.job.config.Job;
import org.elasticsearch.xpack.ml.job.config.MlFilter; import org.elasticsearch.xpack.ml.job.config.MlFilter;
import org.elasticsearch.xpack.ml.job.persistence.JobProvider;
import org.elasticsearch.xpack.ml.job.process.NativeController; import org.elasticsearch.xpack.ml.job.process.NativeController;
import org.elasticsearch.xpack.ml.job.process.ProcessCtrl; import org.elasticsearch.xpack.ml.job.process.ProcessCtrl;
import org.elasticsearch.xpack.ml.job.process.ProcessPipes; import org.elasticsearch.xpack.ml.job.process.ProcessPipes;
@ -44,14 +43,11 @@ public class NativeAutodetectProcessFactory implements AutodetectProcessFactory
private final Client client; private final Client client;
private final Environment env; private final Environment env;
private final Settings settings; private final Settings settings;
private final JobProvider jobProvider;
private final NativeController nativeController; private final NativeController nativeController;
public NativeAutodetectProcessFactory(JobProvider jobProvider, Environment env, Settings settings, public NativeAutodetectProcessFactory(Environment env, Settings settings, NativeController nativeController, Client client) {
NativeController nativeController, Client client) {
this.env = Objects.requireNonNull(env); this.env = Objects.requireNonNull(env);
this.settings = Objects.requireNonNull(settings); this.settings = Objects.requireNonNull(settings);
this.jobProvider = Objects.requireNonNull(jobProvider);
this.nativeController = Objects.requireNonNull(nativeController); this.nativeController = Objects.requireNonNull(nativeController);
this.client = client; this.client = client;
} }