cloud-fan commented on a change in pull request #32718:
URL: https://github.com/apache/spark/pull/32718#discussion_r643063853
##########
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/finishAnalysis.scala
##########
@@ -98,17 +101,21 @@ object ComputeCurrentTime extends Rule[LogicalPlan] {
* Replaces the expression of CurrentDatabase with the current database name.
* Replaces the expression of CurrentCatalog with the current catalog name.
*/
-case class GetCurrentDatabaseAndCatalog(catalogManager: CatalogManager)
extends Rule[LogicalPlan] {
+case class ReplaceCurrentLike(catalogManager: CatalogManager) extends
Rule[LogicalPlan] {
Review comment:
after this renaming, I think current_date etc. should also be calculated
here. We can do it in a followup.
--
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:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
