kfaraz commented on code in PR #18818:
URL: https://github.com/apache/druid/pull/18818#discussion_r2601092138
##########
services/src/test/java/org/apache/druid/testing/embedded/EmbeddedClusterApis.java:
##########
@@ -412,11 +412,11 @@ public SupervisorStatus getSupervisorStatus(String
supervisorId)
// STATIC UTILITY METHODS
/**
- * Creates a random datasource name prefixed with {@link
TestDataSource#WIKI}.
+ * Creates a random datasource name prefixed with {@link
TestDataSource#PREFIX}.
*/
public static String createTestDatasourceName()
{
- return TestDataSource.WIKI + "_" + IdUtils.getRandomId();
+ return TestDataSource.PREFIX + "_" + IdUtils.getRandomId();
Review Comment:
sure, that's reasonable. You may just use the string `datasource`. I think
we can skip adding it as a constant.
--
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]
