Github user mmiklavc commented on a diff in the pull request:
https://github.com/apache/metron/pull/995#discussion_r184145151
--- Diff:
metron-platform/metron-solr/src/test/java/org/apache/metron/solr/integration/SolrUpdateIntegrationTest.java
---
@@ -20,15 +20,21 @@
import java.util.HashMap;
import java.util.List;
import java.util.Map;
+import java.util.Optional;
+
import org.apache.metron.indexing.dao.IndexDao;
+import org.apache.metron.indexing.dao.MultiIndexDao;
import org.apache.metron.indexing.dao.UpdateIntegrationTest;
+import org.apache.metron.indexing.dao.update.Document;
import org.apache.metron.integration.InMemoryComponent;
import org.apache.metron.solr.dao.SolrDao;
import org.apache.metron.solr.integration.components.SolrComponent;
+import org.junit.Assert;
+import org.junit.Test;
public class SolrUpdateIntegrationTest extends UpdateIntegrationTest {
- SolrComponent solrComponent;
+ protected static SolrComponent solrComponent;
--- End diff --
Why `protected static`?
---