MaxGekk opened a new pull request #31635:
URL: https://github.com/apache/spark/pull/31635
### What changes were proposed in this pull request?
Rename the execution node `AlterTableRecoverPartitionsCommand` for the
commands:
- `MSCK REPAIR TABLE table [{ADD|DROP|SYNC} PARTITIONS]`
- `ALTER TABLE table RECOVER PARTITIONS`
to `RepairTableCommand`.
### Why are the changes needed?
1. After the PR https://github.com/apache/spark/pull/31499, `ALTER TABLE
table RECOVER PARTITIONS` is equal to `MSCK REPAIR TABLE table ADD PARTITIONS`.
And mapping of the generic command `MSCK REPAIR TABLE` to the more specific
execution node `AlterTableRecoverPartitionsCommand` can confuse devs in the
future.
2. `ALTER TABLE table RECOVER PARTITIONS` does not support any
options/extensions. So, additional parameters `enableAddPartitions` and
`enableDropPartitions` in `AlterTableRecoverPartitionsCommand` confuse as well.
### Does this PR introduce _any_ user-facing change?
No because this is internal API.
### How was this patch tested?
By running the existing test suites:
```
$ build/sbt -Phive-2.3 -Phive-thriftserver "test:testOnly
*AlterTableRecoverPartitionsSuite"
$ build/sbt "test:testOnly *AlterTableRecoverPartitionsParserSuite"
$ build/sbt -Phive-2.3 -Phive-thriftserver "test:testOnly
*MsckRepairTableSuite"
$ build/sbt "test:testOnly *MsckRepairTableParserSuite"
```
----------------------------------------------------------------
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
