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

ASF GitHub Bot commented on NIFI-3633:
--------------------------------------

Github user JPercivall commented on a diff in the pull request:

    https://github.com/apache/nifi/pull/1610#discussion_r110550470
  
    --- Diff: 
nifi-bootstrap/src/main/java/org/apache/nifi/bootstrap/NotificationServiceManager.java
 ---
    @@ -420,15 +424,15 @@ public static Element getChild(final Element element, 
final String tagName) {
         }
     
         private static class ConfiguredNotificationService {
    -        private final NotificationService service;
    +        private final AbstractNotificationService service;
             private final Map<String, String> properties;
     
    -        public ConfiguredNotificationService(final NotificationService 
service, final Map<String, String> properties) {
    +        public ConfiguredNotificationService(final 
AbstractNotificationService service, final Map<String, String> properties) {
    --- End diff --
    
    See above, reverting.


> Create an HttpNotifierService implementation of the Bootstrap Notification 
> Service
> ----------------------------------------------------------------------------------
>
>                 Key: NIFI-3633
>                 URL: https://issues.apache.org/jira/browse/NIFI-3633
>             Project: Apache NiFi
>          Issue Type: Improvement
>            Reporter: Joseph Percivall
>            Assignee: Joseph Witt
>             Fix For: 1.2.0
>
>
> In order to notify "when the NiFi bootstrap starts or stops NiFi, or detects 
> that it has died unexpectedly" the bootstrap has the notification service but 
> currently there is only one implementation, the EmailNotifierService[1]. 
> An HttpNotifierService should be created in order to more generically post 
> notifications. 
> [1] 
> https://github.com/mcgilman/nifi/blob/3a7ddc6a35f143c1d4a735226fe7467a83d04583/nifi-bootstrap/src/main/java/org/apache/nifi/bootstrap/notification/email/EmailNotificationService.java#L47-L47



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to