cloud-fan commented on a change in pull request #32454:
URL: https://github.com/apache/spark/pull/32454#discussion_r629085064



##########
File path: sql/core/src/test/scala/org/apache/spark/sql/TPCDSBase.scala
##########
@@ -24,7 +24,7 @@ import org.apache.spark.sql.test.SharedSparkSession
 trait TPCDSBase extends SharedSparkSession with TPCDSSchema {
 
   // The TPCDS queries below are based on v1.4
-  val tpcdsQueries = Seq(
+  def tpcdsQueries: Seq[String] = Seq(
     "q1", "q2", "q3", "q4", "q5", "q6", "q7", "q8", "q9", "q10", "q11",

Review comment:
       shall we remove q6 from here for all the tests, if the only difference 
is an extra order by column?




-- 
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:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to