Fix Event Handlers registration Issue (#16544)

* Fix Event Handlers Issue * Review Comments
open-metadata · Jun 5, 2024 · d374e48 · d374e48
1 parent dff0aa8
commit d374e48
Showing 1 changed file with 17 additions and 11 deletions.
diff --git a/openmetadata-service/src/main/java/org/openmetadata/service/events/EventFilter.java b/openmetadata-service/src/main/java/org/openmetadata/service/events/EventFilter.java
@@ -13,6 +13,8 @@
 
 package org.openmetadata.service.events;
 
+import static org.openmetadata.common.utils.CommonUtil.nullOrEmpty;
+
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.HashSet;
@@ -44,22 +46,26 @@ public EventFilter(OpenMetadataApplicationConfig config) {
     registerEventHandlers(config);
   }
 
+  @SuppressWarnings("unchecked")
   private void registerEventHandlers(OpenMetadataApplicationConfig config) {
-    try {
+    if (nullOrEmpty(config.getEventHandlerConfiguration())) {
       Set<String> eventHandlerClassNames =
           new HashSet<>(config.getEventHandlerConfiguration().getEventHandlerClassNames());
       for (String eventHandlerClassName : eventHandlerClassNames) {
-        @SuppressWarnings("unchecked")
-        EventHandler eventHandler =
-            ((Class<EventHandler>) Class.forName(eventHandlerClassName))
-                .getConstructor()
-                .newInstance();
-        eventHandler.init(config);
-        eventHandlers.add(eventHandler);
-        LOG.info("Added event handler {}", eventHandlerClassName);
+        try {
+          EventHandler eventHandler =
+              ((Class<EventHandler>) Class.forName(eventHandlerClassName))
+                  .getConstructor()
+                  .newInstance();
+          eventHandler.init(config);
+          eventHandlers.add(eventHandler);
+          LOG.info("Added event handler {}", eventHandlerClassName);
+        } catch (Exception e) {
+          LOG.info("Exception ", e);
+        }
       }
-    } catch (Exception e) {
-      LOG.info("Exception ", e);
+    } else {
+      LOG.info("Event handler configuration is empty");
     }
   }