Technoboy- commented on issue #1444: move alert module into server package URL: https://github.com/apache/incubator-dolphinscheduler/pull/1444#issuecomment-564870127 > I agree to remove the alert module > > But > > Email, SMS and other alarms can be tools directly, put in common module, define the communication interface between them, and call directly > > I personally think that if you want to modify this, one PR will be done, and multiple PRs will be more complicated. Sure, Mr. Qiao
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
