kfaraz commented on code in PR #18228:
URL: https://github.com/apache/druid/pull/18228#discussion_r2197415474
##########
services/src/test/java/org/apache/druid/testing/embedded/EmbeddedDruidServer.java:
##########
@@ -66,6 +70,34 @@ public abstract class EmbeddedDruidServer<T extends
EmbeddedDruidServer<T>> impl
this.getClass().getSimpleName(),
SERVER_ID.incrementAndGet()
);
+ beforeStartHooks.add(
+ (cluster, self) -> {
+ // Add properties for temporary directories used by the servers
+ final String logsDirectory =
cluster.getTestFolder().getOrCreateFolder("indexer-logs").getAbsolutePath();
+ final String taskDirectory =
cluster.getTestFolder().newFolder().getAbsolutePath();
+ final String storageDirectory =
cluster.getTestFolder().newFolder().getAbsolutePath();
+ log.info(
+ "Server[%s] using directories: task directory[%s], logs
directory[%s], storage directory[%s].",
+ self.getName(),
+ taskDirectory,
+ logsDirectory,
+ storageDirectory
+ );
+ self.addProperty("druid.host", "localhost");
Review Comment:
I see. That makes sense.
I am currently working on something where I think I might need the
non-localhost hostname.
But I will see what I can do to avoid that requirement as it is cleaner to
just have it be `localhost`.
--
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]
