cloud-fan commented on a change in pull request #27249: [SPARK-30019][SQL] Add
the owner property to v2 table
URL: https://github.com/apache/spark/pull/27249#discussion_r368559056
##########
File path:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/HiveDDLSuite.scala
##########
@@ -418,13 +418,26 @@ class HiveCatalogedDDLSuite extends DDLSuite with
TestHiveSingleton with BeforeA
catalog.reset()
}
}
+
+ test("Table Ownership") {
+ val catalog = spark.sessionState.catalog
+ try {
+ sql(s"CREATE TABLE spark_30019(k int)")
+ assert(sql(s"DESCRIBE TABLE EXTENDED
spark_30019").where("col_name='Owner'")
+ .collect().head.getString(1) === Utils.getCurrentUserName())
+ val e2 = intercept[ParseException](
+ sql(s"CREATE TABLE spark_30019_2 WITH
TBLPROPERTIES('owner'='spark_30019')"))
Review comment:
do we need it? I think we already have CREATE/REPLACE/ALTER TABLE tests for
reserved properties.
----------------------------------------------------------------
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]
