Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/10801#discussion_r49973184
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/ParserDialect.scala
---
@@ -17,16 +17,12 @@
package org.apache.spark.sql.catalyst
-import org.apache.spark.annotation.DeveloperApi
import org.apache.spark.sql.catalyst.expressions.Expression
import org.apache.spark.sql.catalyst.plans.logical.LogicalPlan
/**
- * Root class of SQL Parser Dialect, and we don't guarantee the binary
- * compatibility for the future release, let's keep it as the internal
- * interface for advanced user.
+ * Interface for a parser.
*/
-@DeveloperApi
trait ParserDialect {
--- End diff --
+1 on naming it ParserInterface/Parser
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
