Hisoka-X commented on issue #8037:
URL: https://github.com/apache/seatunnel/issues/8037#issuecomment-2481845446

   > Centralized Notification Engine (or extending the existing notification 
system).
   
   we already have `Centralized Notification Engine` named 
[EventListener](https://seatunnel.apache.org/docs/2.3.8/concept/event-listener).
   
   > On events, such as JobCompletion, JobError, JobFailure.
   
   we should add new event type named `JobFinishedEvent`, `JobFailedEvent` etc 
in 
https://github.com/apache/seatunnel/blob/3fb05da365649dddaeaf7cc21e167037b3bd40f6/seatunnel-api/src/main/java/org/apache/seatunnel/api/event/EventType.java#L20.
 Then trigger it on 
https://github.com/apache/seatunnel/blob/3fb05da365649dddaeaf7cc21e167037b3bd40f6/seatunnel-engine/seatunnel-engine-server/src/main/java/org/apache/seatunnel/engine/server/master/JobMaster.java#L130
   
   > Notification listeners have connectors defined in their respective 
configurations.
   
   users can implement theirself event handler 
https://github.com/apache/seatunnel/blob/3fb05da365649dddaeaf7cc21e167037b3bd40f6/seatunnel-api/src/main/java/org/apache/seatunnel/api/event/EventHandler.java#L22
 or reuse 
https://github.com/apache/seatunnel/blob/3fb05da365649dddaeaf7cc21e167037b3bd40f6/seatunnel-engine/seatunnel-engine-server/src/main/java/org/apache/seatunnel/engine/server/event/JobEventHttpReportHandler.java#L49


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to