nuno-faria commented on code in PR #19316:
URL: https://github.com/apache/datafusion/pull/19316#discussion_r2620631441


##########
datafusion/sqllogictest/test_files/explain_analyze.slt:
##########
@@ -25,3 +25,23 @@ Plan with Metrics LazyMemoryExec: partitions=1, 
batch_generators=[generate_serie
 
 statement ok
 reset datafusion.explain.analyze_level;
+
+
+# test auto_explain
+
+statement ok
+set datafusion.explain.auto_explain_output = 
'test_files/scratch/auto_explain.txt';

Review Comment:
   I originally tried to load the file to a table as CSV, as I think it is the 
only feasible way to check the contents, but since the file cannot be removed 
the result would always change. I mainly added these sqllogictests just to 
check the "set ..." commands.
   
   As for removing the file, I'm not sure it is possible. With that said, I 
don't think it is necessary since it's written to the sqllogictest temporary 
dir.



-- 
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]

Reply via email to