Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17433#discussion_r108104968
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -146,6 +141,11 @@ abstract class Optimizer(sessionCatalog:
SessionCatalog, conf: CatalystConf)
s.withNewPlan(newPlan)
}
}
+
+ /**
+ * Override to provide additional rules for the operator optimization
batch.
--- End diff --
Not sure whether we need to split the batch `Operator Optimizations` to
smaller independent batches or move some rules out of this batch in the future.
If so, the location of this rule becomes unstable. We might need to explain it
in the comment.
---
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]
