[GitHub] metron pull request #825: METRON-1290: Only first 10 alerts are update when ...

[cestella]

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

    https://github.com/apache/metron/pull/825#discussion_r149421326
  
    --- Diff: 
metron-platform/metron-elasticsearch/src/test/java/org/apache/metron/elasticsearch/dao/ElasticsearchMetaAlertDaoTest.java
 ---
    @@ -489,4 +505,54 @@ public void testHandleMetaUpdateAlert() throws 
IOException {
         verify(mockEsDao, times(1)).getLatest(guidAlert, null);
         verify(mockEsDao, times(1)).batchUpdate(updates);
       }
    +
    +  @Test
    +  public void getAllAlertsForMetaAlertShouldReturnAllAlerts() throws 
Exception {
    --- End diff --
    
    Agreed


---