imback82 commented on a change in pull request #26921: [SPARK-30282][SQL]
Migrate SHOW TBLPROPERTIES to new framework
URL: https://github.com/apache/spark/pull/26921#discussion_r367781985
##########
File path:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/HiveCommandSuite.scala
##########
@@ -147,16 +147,18 @@ class HiveCommandSuite extends QueryTest with
SQLTestUtils with TestHiveSingleto
checkAnswer(sql("SHOW TBLPROPERTIES parquet_tab2('`prop2Key`')"),
Row("prop2Val"))
}
- test("show tblproperties for spark temporary table - empty row") {
+ test("show tblproperties for spark temporary table - AnalysisException is
thrown") {
withTempView("parquet_temp") {
sql(
"""
|CREATE TEMPORARY VIEW parquet_temp (c1 INT, c2 STRING)
|USING org.apache.spark.sql.parquet.DefaultSource
""".stripMargin)
- // An empty sequence of row is returned for session temporary table.
- checkAnswer(sql("SHOW TBLPROPERTIES parquet_temp"), Nil)
+ val message = intercept[AnalysisException] {
+ sql("SHOW TBLPROPERTIES parquet_temp")
+ }.getMessage
+ assert(message.contains("parquet_temp is a temp view not table"))
Review comment:
Sure, I will add it. Thanks!
----------------------------------------------------------------
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]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
