[ 
https://issues.apache.org/jira/browse/KYLIN-4780?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17912452#comment-17912452
 ] 

ASF GitHub Bot commented on KYLIN-4780:
---------------------------------------

yanghua closed pull request #1596: [KYLIN-4780] Support dingtalk notify
URL: https://github.com/apache/kylin/pull/1596




> Refactor AbstractExecutable to extract the notification relevant business 
> logic into a single component
> -------------------------------------------------------------------------------------------------------
>
>                 Key: KYLIN-4780
>                 URL: https://issues.apache.org/jira/browse/KYLIN-4780
>             Project: Kylin
>          Issue Type: Improvement
>          Components: Integration
>            Reporter: vinoyang
>            Assignee: chenjie
>            Priority: Major
>
> Currently, Kylin supports mail service to know the change around the cube. 
> However, there are some issues about the design, e.g.
>  * AbstractExecutable contains an implementation of the mechanism of a 
> notification listener, but only support mail service, it does not belong 
> there;
>  * The current implementation is hard to scale to other notification e.g. 
> Ding Talk(which is a popular production supports by Alibaba)
> I propose to refactor the current implementation to extract the notification 
> logic and redesign a new common notification/listener



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to