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.


---

Reply via email to