You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by ji...@apache.org on 2019/01/18 22:02:11 UTC
[incubator-pinot] branch master updated: [TE] alerter - do not send
replay anomalies (#3720)
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 <ji...@linkedin.com>
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: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org