Skip to content

Commit 255c54e

Browse files
panbingkunattilapiros
authored andcommitted
[SPARK-49076][SQL] Fix the outdated logical plan name in AstBuilder's comments
### What changes were proposed in this pull request? The pr aims to fix the outdated `logical plan name` in `AstBuilder's` comments. ### Why are the changes needed? - After the pr apache#33609, the name of the logical plan below has been changed: `AlterTableAddColumns` -> `AddColumns` `AlterTableRenameColumn` -> `RenameColumn` `AlterTableAlterColumn` -> `AlterColumn` `AlterTableDropColumns` -> `DropColumns` - After the pr apache#30398 The name of the logical plan `ShowPartitionsStatement` has been changed to `ShowPartitions`. ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? Only update comments. ### Was this patch authored or co-authored using generative AI tooling? No. Closes apache#47562 from panbingkun/fix_astbuilder. Lead-authored-by: panbingkun <[email protected]> Co-authored-by: panbingkun <[email protected]> Signed-off-by: yangjie01 <[email protected]>
1 parent ec8aa90 commit 255c54e

File tree

1 file changed

+8
-8
lines changed
  • sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser

1 file changed

+8
-8
lines changed

sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/AstBuilder.scala

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -61,8 +61,8 @@ import org.apache.spark.util.random.RandomSampler
6161
* The AstBuilder converts an ANTLR4 ParseTree into a catalyst Expression, LogicalPlan or
6262
* TableIdentifier.
6363
*/
64-
class AstBuilder extends DataTypeAstBuilder with SQLConfHelper
65-
with Logging with DataTypeErrorsBase {
64+
class AstBuilder extends DataTypeAstBuilder
65+
with SQLConfHelper with Logging with DataTypeErrorsBase {
6666
import org.apache.spark.sql.connector.catalog.CatalogV2Implicits._
6767
import ParserUtils._
6868

@@ -4452,7 +4452,7 @@ class AstBuilder extends DataTypeAstBuilder with SQLConfHelper
44524452
}
44534453

44544454
/**
4455-
* Parse a [[AlterTableAddColumns]] command.
4455+
* Parse a [[AddColumns]] command.
44564456
*
44574457
* For example:
44584458
* {{{
@@ -4469,7 +4469,7 @@ class AstBuilder extends DataTypeAstBuilder with SQLConfHelper
44694469
}
44704470

44714471
/**
4472-
* Parse a [[AlterTableRenameColumn]] command.
4472+
* Parse a [[RenameColumn]] command.
44734473
*
44744474
* For example:
44754475
* {{{
@@ -4485,7 +4485,7 @@ class AstBuilder extends DataTypeAstBuilder with SQLConfHelper
44854485
}
44864486

44874487
/**
4488-
* Parse a [[AlterTableAlterColumn]] command to alter a column's property.
4488+
* Parse a [[AlterColumn]] command to alter a column's property.
44894489
*
44904490
* For example:
44914491
* {{{
@@ -4555,7 +4555,7 @@ class AstBuilder extends DataTypeAstBuilder with SQLConfHelper
45554555
}
45564556

45574557
/**
4558-
* Parse a [[AlterTableAlterColumn]] command. This is Hive SQL syntax.
4558+
* Parse a [[AlterColumn]] command. This is Hive SQL syntax.
45594559
*
45604560
* For example:
45614561
* {{{
@@ -4639,7 +4639,7 @@ class AstBuilder extends DataTypeAstBuilder with SQLConfHelper
46394639
}
46404640

46414641
/**
4642-
* Parse a [[AlterTableDropColumns]] command.
4642+
* Parse a [[DropColumns]] command.
46434643
*
46444644
* For example:
46454645
* {{{
@@ -4979,7 +4979,7 @@ class AstBuilder extends DataTypeAstBuilder with SQLConfHelper
49794979
* A command for users to list the partition names of a table. If partition spec is specified,
49804980
* partitions that match the spec are returned. Otherwise an empty result set is returned.
49814981
*
4982-
* This function creates a [[ShowPartitionsStatement]] logical plan
4982+
* This function creates a [[ShowPartitions]] logical plan
49834983
*
49844984
* The syntax of using this command in SQL is:
49854985
* {{{

0 commit comments

Comments
 (0)