This is an automated email from the ASF dual-hosted git repository.
akshat pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/druid.git
The following commit(s) were added to refs/heads/master by this push:
new 8ba208f372b Fix quidem output file path for properly handling nested
input files (#17897)
8ba208f372b is described below
commit 8ba208f372b0578c3c4b887b011835975c5b65b8
Author: Akshat Jain <[email protected]>
AuthorDate: Thu Apr 10 20:54:10 2025 +0530
Fix quidem output file path for properly handling nested input files
(#17897)
* Fix quidem output file path for properly handling nested input files
* Delete output file if the testcase passes
---
sql/src/test/java/org/apache/druid/quidem/DruidQuidemTestBase.java | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/sql/src/test/java/org/apache/druid/quidem/DruidQuidemTestBase.java
b/sql/src/test/java/org/apache/druid/quidem/DruidQuidemTestBase.java
index 7088d7b6595..c8fda260274 100644
--- a/sql/src/test/java/org/apache/druid/quidem/DruidQuidemTestBase.java
+++ b/sql/src/test/java/org/apache/druid/quidem/DruidQuidemTestBase.java
@@ -218,7 +218,7 @@ public abstract class DruidQuidemTestBase
final String testName = testConfig.getTestName();
File inFile = new File(getTestRoot(), testConfig.fileName);
- final File outFile = new File(inFile.getParentFile(), testName +
".iq.out");
+ final File outFile = new File(getTestRoot(), testName + ".out");
try (AutoCloseable closeable = Threads.withThreadName(testName)) {
druidQuidemRunner.run(inFile, outFile, testConfig.componentSupplierName);
}
@@ -295,6 +295,10 @@ public abstract class DruidQuidemTestBase
} else {
fail("Files differ: " + outFile + " " + inFile + "\n" + diff);
}
+ } else {
+ if (outFile.exists()) {
+ outFile.delete();
+ }
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
