cryptoe commented on code in PR #12957:
URL: https://github.com/apache/druid/pull/12957#discussion_r955692898
##########
integration-tests-ex/cases/src/test/java/org/apache/druid/testsEx/config/ClusterConfigTest.java:
##########
@@ -85,14 +85,16 @@ public void testYaml() throws FileNotFoundException
assertEquals("http://localhost:8888";, service.clientUrl());
assertEquals("http://localhost:8888";, resolved.routerUrl());
- File userEnv = new File(
- new File(
- System.getProperty("user.home"),
- "druid-it"),
- "Test.env");
- try (PrintWriter out = new PrintWriter(new OutputStreamWriter(new
FileOutputStream(userEnv), StandardCharsets.UTF_8))) {
- out.println("druid_user_var=user");
- }
+ // This test is user-specific. Enable it only if this particular feature
changes.
+ // File userEnv = new File(
Review Comment:
Shouldn't this be removed all together ?
--
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]
