Github user markap14 commented on the issue:

    https://github.com/apache/nifi/pull/2773
  
    @mattyb149 I think this makes sense but have a couple of thoughts on the PR:
    - Probably should use the value set in NiFiProperties for the http/https 
hostname if it is populated. This would be more consistent with the value 
provided to the Cluster Coordinator and also is better for multi-homed 
environments.
    - If not available, I would do the 
`InetAddress.getLocalHost().getHostAddress()` address only once and stash away 
that value in a `final` member variable rather than performing the lookup for 
each Bulletin that is created.


---

Reply via email to