[GitHub] [spark] wzhfy commented on a change in pull request #29636: [SPARK-32786][SQL][TEST] Improve performance for some slow DPP tests

[GitBox <gi...@apache.org>]

wzhfy commented on a change in pull request #29636:
URL: https://github.com/apache/spark/pull/29636#discussion_r482862098



##########
File path: sql/core/src/test/scala/org/apache/spark/sql/DynamicPartitionPruningSuite.scala
##########
@@ -1157,7 +1143,7 @@ abstract class DynamicPartitionPruningSuiteBase
   test("cleanup any DPP filter that isn't pushed down due to expression id clashes") {
     withSQLConf(SQLConf.DYNAMIC_PARTITION_PRUNING_REUSE_BROADCAST_ONLY.key -> "true") {
       withTable("fact", "dim") {
-        spark.range(1000).select($"id".as("A"), $"id".as("AA"))
+        spark.range(20).select($"id".as("A"), $"id".as("AA"))

Review comment:
       We don't have two reasonable join keys between available tables, so I just reduce the records (number of partitions) here. It's unnecessary to create so many partitions in this test.




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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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