alamb commented on issue #9140: URL: https://github.com/apache/arrow-datafusion/issues/9140#issuecomment-1999789252
> @alamb Are you talking about removing cloned or moving short inlist into orlist simplifier I was thinking in general to simplify the problem by moving as many rules as possible into the large `match` statement here https://github.com/apache/arrow-datafusion/blob/81b0a011705b17a09f494f550a5382b0c3414597/datafusion/optimizer/src/simplify_expressions/expr_simplifier.rs#L630 > short inlist into orlist simplifier should be started after the other 3 rules have been done recursively But, it does not seem there is a way to enforce the order in one f_up function. I agree -- f_up gets run on each Expr of the tree so you can't express "apply three rules after this other" So maybe we could move the three rules first into the large `match` and then figure out what to do with the `short inlist into orlist simplifier` rule -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
