spark-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [spark] huaxingao opened a new pull request #26374: [SPARK-29695][SQL] ALTER TABLE (SerDe properties) should look up catalog/table like v2 commands
Date Sun, 03 Nov 2019 00:25:06 GMT
huaxingao opened a new pull request #26374: [SPARK-29695][SQL] ALTER TABLE (SerDe properties)
should look up catalog/table like v2 commands
URL: https://github.com/apache/spark/pull/26374
 
 
   
   ### What changes were proposed in this pull request?
   Add AlterTableSerDePropertiesStatement and make ALTER TABLE ... SET SERDE/SERDEPROPERTIES
go through the same catalog/table resolution framework of v2 commands.
   
   ### Why are the changes needed?
   It's important to make all the commands have the same table resolution behavior, to avoid
confusing end-users. e.g.
   ```
   USE my_catalog
   DESC t // success and describe the table t from my_catalog
   ALTER TABLE t SET SERDE 'org.apache.class' // report table not found as there is no table
t in the session catalog
   ```
   
   ### Does this PR introduce any user-facing change?
   Yes. When running ALTER TABLE ... SET SERDE/SERDEPROPERTIES, Spark fails the command if
the current catalog is set to a v2 catalog, or the table name specified a v2 catalog.
   
   ### How was this patch tested?
   Unit tests.
   

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

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


Mime
View raw message