You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spark.apache.org by do...@apache.org on 2023/06/18 04:10:54 UTC

[spark] branch master updated: [SPARK-44086][SQL][TESTS] Move `DataFrameFunctionsSuite` to `sql - slow`

This is an automated email from the ASF dual-hosted git repository.

dongjoon pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/master by this push:
     new ffba5972f69 [SPARK-44086][SQL][TESTS] Move `DataFrameFunctionsSuite` to `sql - slow`
ffba5972f69 is described below

commit ffba5972f6945d0b655d4f554007baf164d31e36
Author: Ruifeng Zheng <ru...@apache.org>
AuthorDate: Sat Jun 17 21:10:45 2023 -0700

    [SPARK-44086][SQL][TESTS] Move `DataFrameFunctionsSuite` to `sql - slow`
    
    ### What changes were proposed in this pull request?
    Move `DataFrameFunctionsSuite` to `sql - slow`
    
    ### Why are the changes needed?
    1, recently `DataFrameFunctionsSuite` is frequently updated for the new functions, while `sql - other` is quite flaky and it's likely to get stuck or failed before `DataFrameFunctionsSuite` is actually tested (see https://github.com/apache/spark/commit/61a195d33936d6059933de017fa251dac961021e and https://github.com/apache/spark/commit/37f9beaba7e8bdf9f417453d7dab0b7773b55200), so it is hard for the reviewers to judge whether all related tests pass;
    2, `sql - other` is actually much slower than `sql - slow`, so I think it is fine to move a suite to `sql - slow`;
    
    ### Does this PR introduce _any_ user-facing change?
    no, test-only
    
    ### How was this patch tested?
    updated GA
    
    Closes #41643 from zhengruifeng/sql_test_move_dfsuite.
    
    Authored-by: Ruifeng Zheng <ru...@apache.org>
    Signed-off-by: Dongjoon Hyun <do...@apache.org>
---
 .../src/test/scala/org/apache/spark/sql/DataFrameFunctionsSuite.scala   | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/sql/core/src/test/scala/org/apache/spark/sql/DataFrameFunctionsSuite.scala b/sql/core/src/test/scala/org/apache/spark/sql/DataFrameFunctionsSuite.scala
index 6664f8ec932..30583cfd0d0 100644
--- a/sql/core/src/test/scala/org/apache/spark/sql/DataFrameFunctionsSuite.scala
+++ b/sql/core/src/test/scala/org/apache/spark/sql/DataFrameFunctionsSuite.scala
@@ -36,10 +36,12 @@ import org.apache.spark.sql.functions._
 import org.apache.spark.sql.internal.SQLConf
 import org.apache.spark.sql.test.SharedSparkSession
 import org.apache.spark.sql.types._
+import org.apache.spark.tags.ExtendedSQLTest
 
 /**
  * Test suite for functions in [[org.apache.spark.sql.functions]].
  */
+@ExtendedSQLTest
 class DataFrameFunctionsSuite extends QueryTest with SharedSparkSession {
   import testImplicits._
 


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