From 8afde3ee82388f278dd48c684f1ce32b267fe02e Mon Sep 17 00:00:00 2001 From: Aniket Katkar Date: Mon, 29 Jan 2024 12:31:52 +0530 Subject: [PATCH] Minor[WIP]: Fix observability cypress tests (#14907) * modify error alert to get more details on API failure in cypress * Fix Alert Type for postgres * revert changes to show API errors in error toast --------- Co-authored-by: mohitdeuex --- .../org/openmetadata/service/jdbi3/ListFilter.java | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/openmetadata-service/src/main/java/org/openmetadata/service/jdbi3/ListFilter.java b/openmetadata-service/src/main/java/org/openmetadata/service/jdbi3/ListFilter.java index 2fd94c049ae..ab71d19a056 100644 --- a/openmetadata-service/src/main/java/org/openmetadata/service/jdbi3/ListFilter.java +++ b/openmetadata-service/src/main/java/org/openmetadata/service/jdbi3/ListFilter.java @@ -77,9 +77,15 @@ public class ListFilter { private String getEventSubscriptionAlertType() { String alertType = queryParams.get("alertType"); - return alertType == null - ? "" - : String.format("JSON_EXTRACT(json, '$.alertType') = '%s'", alertType); + if (alertType == null) { + return ""; + } else { + if (Boolean.TRUE.equals(DatasourceConfig.getInstance().isMySQL())) { + return String.format("JSON_EXTRACT(json, '$.alertType') = '%s'", alertType); + } else { + return String.format("json->>'alertType' = '%s'", alertType); + } + } } private String getTestCaseResolutionStatusType() {