akpatnam25 commented on code in PR #2633:
URL: https://github.com/apache/celeborn/pull/2633#discussion_r1686865105


##########
docs/configuration/network.md:
##########
@@ -42,6 +42,7 @@ license: |
 | celeborn.<role>.rpc.dispatcher.threads | <value of 
celeborn.rpc.dispatcher.threads> | false | Threads number of message 
dispatcher event loop for roles |  |  | 
 | celeborn.io.maxDefaultNettyThreads | 64 | false | Max default netty threads 
| 0.3.2 |  | 
 | celeborn.network.bind.preferIpAddress | true | false | When `true`, prefer 
to use IP address, otherwise FQDN. This configuration only takes effects when 
the bind hostname is not set explicitly, in such case, Celeborn will find the 
first non-loopback address to bind. | 0.3.0 |  | 
+| celeborn.network.bind.wildcardAddress | false | false | When `true`, the 
bind address will be set to a wildcard address, while the advertise address 
will remain as whatever is set by `celeborn.network.bind.preferIpAddress`. This 
is helpful in dual-stack environments, where the service must listen to both 
IPv4 and IPv6 clients. | 0.6.0 |  | 

Review Comment:
   @pan3793 no this PR just configures allows RPC and HTTP servers to bind to 
wildcard address. 
   Initially, when `celeborn.network.bind.preferIpAddress` is set, the bind 
address and advertise address are both determined by 
`celeborn.network.bind.preferIpAddress` (either ip address or hostname for 
both). Now, if  `celeborn.network.bind.wildcardAddress` is set, advertise 
address will still be either ip address or hostname based on 
`celeborn.network.bind.preferIpAddress`, but bind address will be wildcard 
address. 
   
   hope this clarifies :) 



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