[SPARK-29612][SQL] ALTER TABLE (RECOVER PARTITIONS) should look up catalog/table like v2 commands

[huaxingao]

What changes were proposed in this pull request?

Add AlterTableRecoverPartitionsStatement and make ALTER TABLE ... RECOVER PARTITIONS 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 RECOVER PARTITIONS  // 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 ... RECOVER PARTITIONS 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.

[SparkQA]

Test build #112722 has finished for PR 26269 at commit e61f6bb.

• This patch fails Spark unit tests.
• This patch merges cleanly.
• This patch adds the following public classes (experimental):
• case class AlterTableRecoverPartitionsStatement(

[huaxingao]

retest this please

[SparkQA]

Test build #112725 has finished for PR 26269 at commit e61f6bb.

• This patch fails Spark unit tests.
• This patch merges cleanly.
• This patch adds the following public classes (experimental):
• case class AlterTableRecoverPartitionsStatement(

[huaxingao]

@viirya Could you please review? The unit tests failed, but it doesn't seem to be related to my changes.

[viirya]

retest this please

[viirya]

The description is not correct at all, SHOW CREATE TABLE t

[viirya]

cc @cloud-fan @imback82

[SparkQA]

Test build #112799 has finished for PR 26269 at commit e61f6bb.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds the following public classes (experimental):
• case class AlterTableRecoverPartitionsStatement(

[SparkQA]

Test build #112804 has finished for PR 26269 at commit aaa6bfd.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds no public classes.

[cloud-fan]

thanks, merging to master!

[huaxingao]

Thanks! @cloud-fan @viirya

[imback82]

Late +1