dongjoon-hyun commented on code in PR #47976:
URL: https://github.com/apache/spark/pull/47976#discussion_r1745735648
##########
sql/core/src/test/scala/org/apache/spark/sql/internal/CatalogSuite.scala:
##########
@@ -700,7 +700,7 @@ class CatalogSuite extends SharedSparkSession with
AnalysisTest with BeforeAndAf
val description = "this is a test table"
withTable("t") {
- withTempDir { dir =>
+ withTempDir(prefix = "test%prefix") { dir =>
Review Comment:
It's orthogonal from this bug fix, isn't it? I'd recommend to proceed that
test utility contribution as an independent PR, @cstavr . We can review
separately.
> That would also work, but I think is useful to have it since we should
expand more on testing of paths with special chars. In the future we could also
change the default value, so this helper will be needed to avoid the special
chars.
--
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: reviews-unsubscr...@spark.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
