rkrishn7 commented on issue #7765: URL: https://github.com/apache/datafusion/issues/7765#issuecomment-2654922651
From a user perspective, my opinion is that representing wildcard with `None` is too implicit. As @alamb mentioned, having an `expr_fn` helper makes this a bit better, but still not great IMO. > As long as we have alternative representation of wildcard (i.e. None in projection expressions) How would we differentiate qualified vs. unqualified wildcards with `None`? Just to think about alternatives, what if we kept `Expr::Wildcard` around and guarantee that it is expanded in any `LogicalPlan`s built via `LogicalPlanBuilder` (related to #14618)?. We could also think about a type safe way to represent this, but I think that would come with significant breaking changes. -- 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]
