[GitHub] [spark] cloud-fan commented on issue #27249: [SPARK-30019][SQL] Add ALTER TABLE SET OWNER syntax

cloud-fan commented on issue #27249: [SPARK-30019][SQL] Add ALTER TABLE SET OWNER syntax URL: https://github.com/apache/spark/pull/27249#issuecomment-576163408 can we add some tests to make sure CREATE/ALTER TABLE fails if setting `owner` property? Also let's update the PR title.

[GitHub] [spark] cloud-fan commented on issue #27249: [SPARK-30019][SQL] Add ALTER TABLE SET OWNER syntax

cloud-fan commented on issue #27249: [SPARK-30019][SQL] Add ALTER TABLE SET OWNER syntax URL: https://github.com/apache/spark/pull/27249#issuecomment-575645343 LGTM This is an automated message from the Apache Git Service.

[GitHub] [spark] cloud-fan commented on issue #27249: [SPARK-30019][SQL] Add ALTER TABLE SET OWNER syntax

cloud-fan commented on issue #27249: [SPARK-30019][SQL] Add ALTER TABLE SET OWNER syntax URL: https://github.com/apache/spark/pull/27249#issuecomment-575645279 I mean, we should follow `DESC NAMESPACE` to implement it, e.g. display `Owner Name: abc` which looks nicer. Anyway we can do it

[GitHub] [spark] cloud-fan commented on issue #27249: [SPARK-30019][SQL] Add ALTER TABLE SET OWNER syntax

cloud-fan commented on issue #27249: [SPARK-30019][SQL] Add ALTER TABLE SET OWNER syntax URL: https://github.com/apache/spark/pull/27249#issuecomment-575599323 shall we also print owner name/type in DESC TABLE like DESC NAMESPACE?