value

ulysses-you · ulysses-you · commit 271a2fb7ed09 · 2021-03-04T20:46:26.000+08:00
diff --git a/sql/core/src/test/scala/org/apache/spark/sql/execution/SQLViewTestSuite.scala b/sql/core/src/test/scala/org/apache/spark/sql/execution/SQLViewTestSuite.scala
@@ -283,12 +283,13 @@ abstract class SQLViewTestSuite extends QueryTest with SQLTestUtils {
   test("SPARK-34613: Fix view does not capture disable hint config") {
     withSQLConf(DISABLE_HINTS.key -> "true") {
       withView("v1") {
-        createView("v1", "SELECT /*+ repartition(1) */ 1")
+        val viewName = createView("v1", "SELECT /*+ repartition(1) */ 1")
         assert(
           sql("SELECT * FROM v1").queryExecution.analyzed.collect {
             case e: Repartition => e
           }.isEmpty
         )
+        checkViewOutput(viewName, Seq(Row(1)))
       }
     }
   }

Original file line number	Diff line number	Diff line change
`@@ -283,12 +283,13 @@ abstract class SQLViewTestSuite extends QueryTest with SQLTestUtils {`
`283`	`283`	`test("SPARK-34613: Fix view does not capture disable hint config") {`
`284`	`284`	`withSQLConf(DISABLE_HINTS.key -> "true") {`
`285`	`285`	`withView("v1") {`
`286`		`- createView("v1", "SELECT /+ repartition(1) / 1")`
	`286`	`+ val viewName = createView("v1", "SELECT /+ repartition(1) / 1")`
`287`	`287`	`assert(`
`288`	`288`	`sql("SELECT * FROM v1").queryExecution.analyzed.collect {`
`289`	`289`	`case e: Repartition => e`
`290`	`290`	`}.isEmpty`
`291`	`291`	`)`
	`292`	`+ checkViewOutput(viewName, Seq(Row(1)))`
`292`	`293`	`}`
`293`	`294`	`}`
`294`	`295`	`}`