You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by gatorsmile <gi...@git.apache.org> on 2018/10/05 21:58:49 UTC

[GitHub] spark pull request #22642: [SPARK-25653][TEST] Add tag ExtendedHiveTest for ...

Github user gatorsmile commented on a diff in the pull request:

    https://github.com/apache/spark/pull/22642#discussion_r223148975
  
    --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/HiveSparkSubmitSuite.scala ---
    @@ -33,11 +33,13 @@ import org.apache.spark.sql.execution.command.DDLUtils
     import org.apache.spark.sql.expressions.Window
     import org.apache.spark.sql.hive.test.{TestHive, TestHiveContext}
     import org.apache.spark.sql.types.{DecimalType, StructType}
    +import org.apache.spark.tags.ExtendedHiveTest
     import org.apache.spark.util.{ResetSystemProperties, Utils}
     
     /**
      * This suite tests spark-submit with applications using HiveContext.
      */
    +@ExtendedHiveTest
    --- End diff --
    
    With this tag, the test suite will still run if we do a code change in sql/hive. Right? cc @cloud-fan @vanzin 


---

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