bitflicker64 commented on code in PR #3149:
URL: https://github.com/apache/hugegraph/pull/3149#discussion_r3852746430


##########
docker/docker-compose-3pd-3store-3server.yml:
##########
@@ -187,16 +207,25 @@ services:
     <<: *server-common
     container_name: hg-server0
     hostname: server0
+    environment:
+      <<: *server-env
+      HG_SERVER_REST_URL: http://server0:8080

Review Comment:
   Correction to my earlier reply on this thread: the fix I described does not 
work and has been removed in 96160375.
   
   `HG_SERVER_REST_URL` becomes `restserver.url`, which is the address the REST 
server binds, not only the one registered with PD. Running the cluster with the 
documented override crash-looped that replica on `java.net.BindException: 
Cannot assign requested address` while the two default replicas stayed healthy.
   
   Your original point stands: this needs a separate advertised-address setting 
in the Server. The README now documents the limitation and points external 
clients at the published ports instead of PD discovery.



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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to