GabrielCWT commented on PR #18692:
URL: https://github.com/apache/druid/pull/18692#issuecomment-3459521663

   Hi @kfaraz, when implementing the embedded test, I encountered a strange bug 
which I could not pinpoint the issue. I'm hoping you have more insight on this 
issue.
   
   When sending the `/status/properties` request to the coordinator's 
`localhost:8081`, it would return the broker's properties.  I confirmed that 
the coordinator was initialising it's own StatusResource and Properties object. 
   
   It seems like the coordinator is hitting the broker's endpoint even though I 
am using `localhost:8081`. 
   
   When I added a router server, the coordinator would return the router's 
properties instead.
   
   ```
     protected EmbeddedDruidCluster createCluster()
     {
       return EmbeddedDruidCluster
           .withZookeeper()
           .addServer(new EmbeddedCoordinator())
           .addServer(overlord)
           .addServer(broker)
           .addServer(new EmbeddedRouter()) // added here
           .addCommonProperty("commonProperty", "commonValue");
     }
     ```
   
   Note: Currently I have omitted verifying the coordinator's properties in the 
embedded test for the reason above.


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