This is an automated email from the ASF dual-hosted git repository.

jihao pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git


The following commit(s) were added to refs/heads/master by this push:
     new f09db48  [TE] alerter - do not send replay anomalies (#3720)
f09db48 is described below

commit f09db48ebf4b0bd7a551e8f0c2fd86c1177e4f7a
Author: Jihao Zhang <[email protected]>
AuthorDate: Fri Jan 18 14:02:06 2019 -0800

    [TE] alerter - do not send replay anomalies (#3720)
    
    Don't send an alert if the anomaly is from replay or user report.
---
 .../pinot/thirdeye/detection/alert/StatefulDetectionAlertFilter.java  | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git 
a/thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/detection/alert/StatefulDetectionAlertFilter.java
 
b/thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/detection/alert/StatefulDetectionAlertFilter.java
index ec12552..21bea17 100644
--- 
a/thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/detection/alert/StatefulDetectionAlertFilter.java
+++ 
b/thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/detection/alert/StatefulDetectionAlertFilter.java
@@ -21,6 +21,7 @@ package org.apache.pinot.thirdeye.detection.alert;
 
 import com.google.common.base.Predicate;
 import com.google.common.collect.Collections2;
+import org.apache.pinot.thirdeye.constant.AnomalyResultSource;
 import org.apache.pinot.thirdeye.datalayer.dto.DetectionAlertConfigDTO;
 import org.apache.pinot.thirdeye.datalayer.dto.MergedAnomalyResultDTO;
 import org.apache.pinot.thirdeye.detection.spi.model.AnomalySlice;
@@ -69,7 +70,8 @@ public abstract class StatefulDetectionAlertFilter extends 
DetectionAlertFilter
               return mergedAnomalyResultDTO != null
                   && !mergedAnomalyResultDTO.isChild()
                   && !AlertUtils.hasFeedback(mergedAnomalyResultDTO)
-                  && (mergedAnomalyResultDTO.getId() == null || 
mergedAnomalyResultDTO.getId() >= minId);
+                  && (mergedAnomalyResultDTO.getId() == null || 
mergedAnomalyResultDTO.getId() >= minId)
+                  && 
mergedAnomalyResultDTO.getAnomalyResultSource().equals(AnomalyResultSource.DEFAULT_ANOMALY_DETECTION);
             }
           });
 


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to