jackwener commented on code in PR #9954:
URL: https://github.com/apache/arrow-datafusion/pull/9954#discussion_r1560549931
##########
datafusion/optimizer/src/optimizer.rs:
##########
@@ -80,12 +84,31 @@ pub trait OptimizerRule {
/// A human readable name for this optimizer rule
fn name(&self) -> &str;
- /// How should the rule be applied by the optimizer? See comments on
[`ApplyOrder`] for details.
+ /// How should the rule be applied by the optimizer? See comments on
+ /// [`ApplyOrder`] for details.
///
- /// If a rule use default None, it should traverse recursively plan inside
itself
+ /// If returns `None`, the default, the rule must handle recursion itself
fn apply_order(&self) -> Option<ApplyOrder> {
None
Review Comment:
> I think most of the cases are BottomUp by default?
we can't assume it.
--
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.
To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
