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

    https://github.com/apache/nifi/pull/386#discussion_r62165158
  
    --- Diff: 
nifi-bootstrap/src/main/java/org/apache/nifi/bootstrap/RunNiFi.java ---
    @@ -324,27 +324,46 @@ private void registerNotificationServices(final 
NotificationServiceManager manag
             defaultLogger.info("Registered {} Notification Services for 
Notification Type {}", registered, type);
         }
     
    -    File getStatusFile() {
    +    File getStatusFile() throws IOException{
    --- End diff --
    
    Default method is used by the ShutdownHook class which is in the same 
package and allows it to not have to provide a logger. Not sure why the 
overloaded method is public (was introduced  early on in NIFI-145, default 
method came later) since it appears to only be used by RunNifi. I will research 
to see if the public method can really because private/protected.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to