Max Gekk 2b76e6d15c [SPARK-34301][SQL] Use logical plan of alter table in CatalogImpl.recoverPartitions() ... ### What changes were proposed in this pull request? Replace v1 exec node `AlterTableRecoverPartitionsCommand` by the logical node `AlterTableRecoverPartitions` in `CatalogImpl.recoverPartitions()`. ### Why are the changes needed? 1. Print user friendly error message for views: ``` my_temp_table is a temp view. 'recoverPartitions()' expects a table ``` Before the changes: ``` Table or view 'my_temp_table' not found in database 'default' ``` 2. To not bind to v1 `ALTER TABLE .. RECOVER PARTITIONS`, and to support v2 tables potentially as well. ### Does this PR introduce _any_ user-facing change? Yes, it can. ### How was this patch tested? By running new test in `CatalogSuite`: ``` $ build/sbt -Phive -Phive-thriftserver "test:testOnly org.apache.spark.sql.internal.CatalogSuite" ``` Closes #31403 from MaxGekk/catalogimpl-recoverPartitions. Authored-by: Max Gekk <max.gekk@gmail.com> Signed-off-by: Wenchen Fan <wenchen@databricks.com>		2021-02-01 14:09:40 +00:00
..
benchmarks	[SPARK-34192][SQL] Move char padding to write side and remove length check on read side too	2021-01-26 02:08:35 +08:00
src	[SPARK-34301][SQL] Use logical plan of alter table in CatalogImpl.recoverPartitions()	2021-02-01 14:09:40 +00:00
pom.xml	[SPARK-33662][BUILD] Setting version to 3.2.0-SNAPSHOT	2020-12-04 14:10:42 -08:00