Github user nickwallen commented on a diff in the pull request:

    https://github.com/apache/metron/pull/1009#discussion_r187162573
  
    --- Diff: 
metron-platform/metron-parsers/src/test/java/org/apache/metron/writers/integration/WriterBoltIntegrationTest.java
 ---
    @@ -88,9 +108,7 @@ public void initialize(Map<String, Object> 
validationConfig, Map<String, Object>
       public static String parserConfigJSON;
     
       @Test
    -  public void test() throws UnableToStartException, IOException, 
ParseException {
    -
    -    UnitTestHelper.setLog4jLevel(CSVParser.class, 
org.apache.log4j.Level.FATAL);
    +  public void 
parser_with_global_validations_writes_bad_records_to_error_topic() throws 
Exception {
    --- End diff --
    
    Maybe just a brief javadoc would help here.  I have to read a lot of code 
to understand what each of these tests are trying to do. One sentence of 
javadoc would save me some time.


---

Reply via email to