This is an automated email from the ASF dual-hosted git repository.
akshayrai09 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 ada802e [TE] API fix for partially migrated alerts in dashboard
(#3801)
ada802e is described below
commit ada802e64a49a4ce3abba8f1ea59a37c0462888c
Author: Akshay Rai <[email protected]>
AuthorDate: Wed Feb 6 17:10:28 2019 -0800
[TE] API fix for partially migrated alerts in dashboard (#3801)
---
.../thirdeye/dashboard/resources/v2/UserDashboardResource.java | 7 +++++++
1 file changed, 7 insertions(+)
diff --git
a/thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/dashboard/resources/v2/UserDashboardResource.java
b/thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/dashboard/resources/v2/UserDashboardResource.java
index 8947e73..377863e 100644
---
a/thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/dashboard/resources/v2/UserDashboardResource.java
+++
b/thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/dashboard/resources/v2/UserDashboardResource.java
@@ -181,6 +181,13 @@ public class UserDashboardResource {
}
}
+ // TODO: deprecate after migration
+ // Support for partially migrated alerts.
+ List<DetectionAlertConfigDTO> notifications =
detectionAlertDAO.findByPredicate(Predicate.EQ("application", application));
+ for (DetectionAlertConfigDTO notification : notifications) {
+ applicationFunctionIds.addAll(notification.getVectorClocks().keySet());
+ }
+
// group (indirect)
Set<Long> groupFunctionIds = new HashSet<>();
if (StringUtils.isNotBlank(group)) {
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
