opendev/gerrit
Code Issues Proposed changes

Remove injection annotations from constructor in AbstractElasticIndex

Browse Source 
This constructor is not called by Guice.

Change-Id: Ida5248659bdc112f6e171759eec780e591989b5f
This commit is contained in:
Hugo Arès
2016-11-15 13:40:24 -08:00
parent 5cca4e5d28
commit 3b943ab0e8
1 changed files with 1 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
gerrit-elasticsearch/src/main/java/com/google/gerrit/elasticsearch/AbstractElasticIndex.java
View File

@@ -27,8 +27,6 @@ import com.google.gerrit.server.index.Index;
import com.google.gerrit.server.index.IndexUtils; import com.google.gerrit.server.index.IndexUtils;
import com.google.gerrit.server.index.Schema; import com.google.gerrit.server.index.Schema;
import com.google.gerrit.server.index.Schema.Values; import com.google.gerrit.server.index.Schema.Values;
import com.google.inject.Inject;
import com.google.inject.assistedinject.Assisted;
import org.eclipse.jgit.lib.Config; import org.eclipse.jgit.lib.Config;
import org.elasticsearch.common.xcontent.XContentBuilder; import org.elasticsearch.common.xcontent.XContentBuilder;
@@ -60,11 +58,10 @@ abstract class AbstractElasticIndex<K, V> implements Index<K, V> {
protected final JestHttpClient client; protected final JestHttpClient client;
@Inject
AbstractElasticIndex(@GerritServerConfig Config cfg, AbstractElasticIndex(@GerritServerConfig Config cfg,
FillArgs fillArgs, FillArgs fillArgs,
SitePaths sitePaths, SitePaths sitePaths,
@Assisted Schema<V> schema) { Schema<V> schema) {
this.fillArgs = fillArgs; this.fillArgs = fillArgs;
this.sitePaths = sitePaths; this.sitePaths = sitePaths;
this.schema = schema; this.schema = schema;