Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/1247#discussion_r236652983 --- Diff: metron-platform/metron-solr/src/test/java/org/apache/metron/solr/integration/SolrUpdateIntegrationTest.java --- @@ -186,4 +195,114 @@ public void testHugeErrorFields() throws Exception { exception.expectMessage("Document contains at least one immense term in field=\"error_hash\""); getDao().update(errorDoc, Optional.of("error")); } + + @Test + @Override + public void test() throws Exception { --- End diff -- This test was previously shared in UpdateIntegrationTest between ES and Solr. With these changes, the tests don't behave exactly the same anymore. That being said, what I did here doesn't look right. I'll dig into this and see what is going on.
---