Skip to content

[SPARK-49076][SQL] Fix the outdated logical plan name in AstBuilder's comments #47562

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
panbingkun wants to merge 2 commits into from
Closed

[SPARK-49076][SQL] Fix the outdated logical plan name in AstBuilder's comments #47562

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
fd06c46
[SPARK-49076][SQL] Fix the outdated logical plan name in 's comments
panbingkun Aug 1, 2024
cc53799
Update sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/pars…
panbingkun Aug 1, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 8 additions & 8 deletions sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/AstBuilder.scala
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -61,8 +61,8 @@ import org.apache.spark.util.random.RandomSampler
* The AstBuilder converts an ANTLR4 ParseTree into a catalyst Expression, LogicalPlan or
* TableIdentifier.
*/
class AstBuilder extends DataTypeAstBuilder with SQLConfHelper
with Logging with DataTypeErrorsBase {
class AstBuilder
extends DataTypeAstBuilder with SQLConfHelper with Logging with DataTypeErrorsBase {
import org.apache.spark.sql.connector.catalog.CatalogV2Implicits._
import ParserUtils._

Expand Down Expand Up @@ -4452,7 +4452,7 @@ class AstBuilder extends DataTypeAstBuilder with SQLConfHelper
}

/**
* Parse a [[AlterTableAddColumns]] command.
* Parse a [[AddColumns]] command.
*
* For example:
* {{{
Expand All @@ -4469,7 +4469,7 @@ class AstBuilder extends DataTypeAstBuilder with SQLConfHelper
}

/**
* Parse a [[AlterTableRenameColumn]] command.
* Parse a [[RenameColumn]] command.
*
* For example:
* {{{
Expand All @@ -4485,7 +4485,7 @@ class AstBuilder extends DataTypeAstBuilder with SQLConfHelper
}

/**
* Parse a [[AlterTableAlterColumn]] command to alter a column's property.
* Parse a [[AlterColumn]] command to alter a column's property.
*
* For example:
* {{{
Expand Down Expand Up @@ -4555,7 +4555,7 @@ class AstBuilder extends DataTypeAstBuilder with SQLConfHelper
}

/**
* Parse a [[AlterTableAlterColumn]] command. This is Hive SQL syntax.
* Parse a [[AlterColumn]] command. This is Hive SQL syntax.
*
* For example:
* {{{
Expand Down Expand Up @@ -4639,7 +4639,7 @@ class AstBuilder extends DataTypeAstBuilder with SQLConfHelper
}

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