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

    https://github.com/apache/metron/pull/811#discussion_r146827154
  
    --- Diff: 
metron-platform/metron-elasticsearch/src/main/java/org/apache/metron/elasticsearch/dao/ElasticsearchMetaAlertDao.java
 ---
    @@ -295,19 +341,199 @@ protected Document 
buildCreateDocument(MultiGetResponse multiGetResponse, List<S
       /**
        * Process an update to a meta alert itself.
        * @param update The update Document to be applied
    -   * @param index The optional index to update to
        * @throws IOException If there's a problem running the update
        */
    -  protected void handleMetaUpdate(Document update, Optional<String> index) 
throws IOException {
    -    // We have an update to a meta alert itself
    -    // If we've updated the alerts field (i.e add/remove), recalculate 
meta alert scores.
    +  protected void handleMetaUpdate(Document update) throws IOException {
    --- End diff --
    
    I'll try to reproduce. I'm not sure what the potential cause could be.  
Could you see if the alerts you added have the properly populated "metaalerts" 
field in the original alert?  It's possible an issue occurs during that update, 
but I don't really have any evidence in any direction.


---

Reply via email to