kfaraz commented on code in PR #18413:
URL: https://github.com/apache/druid/pull/18413#discussion_r2293528368
##########
extensions-core/druid-testcontainers/src/main/java/org/apache/druid/testing/DruidCommand.java:
##########
@@ -164,7 +165,12 @@ enum Server implements DruidCommand
private final Integer[] exposedPorts;
private final Map<String, String> defaultProperties;
- Server(String name, String javaOpts, Map<String, String>
defaultProperties, Integer... exposedPorts)
+ Server(
+ String name,
+ String javaOpts,
+ Map<String, String> defaultProperties,
+ Integer... exposedPorts
+ )
Review Comment:
Please revert all the formatting changes to this class as well.
##########
embedded-tests/src/test/java/org/apache/druid/testing/embedded/docker/CustomNodeRoleDockerTest.java:
##########
@@ -89,5 +89,17 @@ public Map<String, String> getDefaultProperties()
{
return Map.of();
}
+
+ @Override
+ public Integer getExposedOperatorPort()
+ {
+ return 9302;
+ }
+
+ @Override
+ public Map<String, String> getOperatorConfiguration()
+ {
+ return Map.of();
+ }
Review Comment:
Should not be needed anymore.
--
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]
