neilconway commented on code in PR #22313:
URL: https://github.com/apache/datafusion/pull/22313#discussion_r3267549626
##########
datafusion/optimizer/src/scalar_subquery_to_join.rs:
##########
@@ -156,14 +156,12 @@ impl OptimizerRule for ScalarSubqueryToJoin {
#[allow(clippy::allow_attributes, clippy::mutable_key_type)]
// Expr contains Arc with interior mutability but is
intentionally used as hash key
let mut expr_to_rewrite_expr_map = HashMap::new();
- #[allow(clippy::allow_attributes, clippy::mutable_key_type)]
- // Expr contains Arc with interior mutability but is
intentionally used as hash key
- let mut subquery_to_expr_map = HashMap::new();
+ let mut alias_to_expr_map: HashMap<String, Expr> =
HashMap::new();
Review Comment:
Good idea! I've implemented this, it's a nice simplification.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
