stevedlawrence commented on a change in pull request #5746:
URL: https://github.com/apache/nifi/pull/5746#discussion_r803941563



##########
File path: 
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-resources/src/main/resources/conf/nifi.properties
##########
@@ -244,6 +244,9 @@ 
nifi.security.user.saml.http.client.read.timeout=${nifi.security.user.saml.http.
 # nifi.security.group.mapping.value.anygroup=$1
 # nifi.security.group.mapping.transform.anygroup=LOWER
 
+# listener bootstrap properties
+nifi.listener.bootstrap.port=${nifi.listener.bootstrap.port}

Review comment:
       My understanding is that they must be different.
   
   Bootstrap will start and listen on `nifi.bootstrap.listen.port`. It will 
then start the main NiFi process and pass an argument that tells that main NiFi 
process what port Bootstrap is listening on.
   
   Then when NiFi starts, it will start listening on 
`nifi.listener.bootstrap.port`, and send a message to Bootstrap (to the port 
bootstrap is listening on, i.e. `nifi.bootstrap.listen.port`) telling it what 
this port it started listening on. At this point, each process is listening on 
a different port and knows which port the other is listening on to send 
commands.
   
   By default these ports are random ephemeral ports, hence this communication 
needed to tell each other what the ports the other is listening on. This patch 
just makes it so you can optionally specify non-random ports.




-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to