Github user tone-zhang commented on the issue:

    https://github.com/apache/spark/pull/14894
  
    @srowen I checked DDLSuite.scala, the case "Create Database using Default 
Warehouse Path" is used to test the default "spark-warehouse" setting. In the 
doc, the default path should be "("user.dir")/spark-warehouse". I prefer to 
keep the path, just remove it after finish the case manually. 
    For the "tempDir" in DDLSuit.scala, I confirm it is point to the temporary 
path. It is safe enough.
    When check  the cases in DDLSuite.scala, I found the "spark-warehouse" is 
always created by the UT frame. At the final, the directory is empty and it 
will not negative impact on the UT. I tried to set the WAREHOUSE_PATH of 
SQLConf to redirect to the temporary path, but failed. May I delete the 
"spark-warehouse" in function afterEach()? Or could you please give me some 
suggestion? Thanks a lot!


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to