ODIn/spark-instrumented-optimizer
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

[SPARK-33838][SQL][DOCS] Comment the PURGE option in the DropTable and in AlterTableDropPartition commands

Browse source 
### What changes were proposed in this pull request?
Add comments for the `PURGE` option to the logical nodes `DropTable` and `AlterTableDropPartition`.

### Why are the changes needed?
To improve code maintenance.

### Does this PR introduce _any_ user-facing change?
No

### How was this patch tested?
By running `./dev/scalastyle`

Closes #30837 from MaxGekk/comment-purge-logical-node.

Authored-by: Max Gekk <max.gekk@gmail.com>
Signed-off-by: Wenchen Fan <wenchen@databricks.com>
This commit is contained in:
Max Gekk 2020-12-21 14:06:31 +00:00 committed by Wenchen Fan
parent 69aa727ff4
commit 661ac10901
1 changed files with 12 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/v2Commands.scala
View file

@ -410,6 +410,14 @@ case class Assignment(key: Expression, value: Expression) extends Expression wit
/**
* The logical plan of the DROP TABLE command.
*
* If the `PURGE` option is set, the table catalog must remove table data by skipping the trash
* even when the catalog has configured one. The option is applicable only for managed tables.
*
* The syntax of this command is:
* {{{
* DROP TABLE [IF EXISTS] table [PURGE];
* }}}
*/
case class DropTable(
child: LogicalPlan,
@ -657,9 +665,12 @@ case class AlterTableAddPartition(
* The logical plan of the ALTER TABLE DROP PARTITION command.
* This may remove the data and metadata for this partition.
*
* If the `PURGE` option is set, the table catalog must remove partition data by skipping the trash
* even when the catalog has configured one. The option is applicable only for managed tables.
*
* The syntax of this command is:
* {{{
* ALTER TABLE table DROP [IF EXISTS] PARTITION spec1[, PARTITION spec2, ...];
* ALTER TABLE table DROP [IF EXISTS] PARTITION spec1[, PARTITION spec2, ...] [PURGE];
* }}}
*/
case class AlterTableDropPartition(