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

    https://github.com/apache/metron/pull/843#discussion_r151799957
  
    --- Diff: 
metron-interface/metron-rest/src/test/java/org/apache/metron/rest/controller/SearchControllerIntegrationTest.java
 ---
    @@ -132,6 +132,30 @@ public void testDefaultQuery() throws Exception {
         sensorIndexingConfigService.delete("bro");
       }
     
    +  @Test
    +  public void testDefaultColumnMetadata() throws Exception {
    --- End diff --
    
    Would you mind adding a comment somewhere that describes in more detail 
what this test case is testing?  Just looking at this test case in isolation, 
it is not clear to me what we are testing.


---

Reply via email to