You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@datafusion.apache.org by "rkrishn7 (via GitHub)" <gi...@apache.org> on 2025/02/12 21:14:09 UTC
[I] Move `ExpandWildcardRule` into Logical Plan construction [datafusion]
rkrishn7 opened a new issue, #14634:
URL: https://github.com/apache/datafusion/issues/14634
### Is your feature request related to a problem or challenge?
Related to #14618.
The first step in integrating analyzer rules in the builder stage is moving `ExpandWildcardRule` out of there and into the builder phase.
### Describe the solution you'd like
_No response_
### Describe alternatives you've considered
_No response_
### Additional context
_No response_
--
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-unsubscribe@datafusion.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org
Re: [I] Move `ExpandWildcardRule` into Logical Plan construction [datafusion]
Posted by "goldmedal (via GitHub)" <gi...@apache.org>.
goldmedal closed issue #14634: Move `ExpandWildcardRule` into Logical Plan construction
URL: https://github.com/apache/datafusion/issues/14634
--
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-unsubscribe@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org
Re: [I] Move `ExpandWildcardRule` into Logical Plan construction [datafusion]
Posted by "jayzhan211 (via GitHub)" <gi...@apache.org>.
jayzhan211 commented on issue #14634:
URL: https://github.com/apache/datafusion/issues/14634#issuecomment-2655380153
Even though `Expr::Wildcard` has been removed, we still need an equivalent concept and a way to "expand wildcard"
--
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-unsubscribe@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org
Re: [I] Move `ExpandWildcardRule` into Logical Plan construction [datafusion]
Posted by "rkrishn7 (via GitHub)" <gi...@apache.org>.
rkrishn7 commented on issue #14634:
URL: https://github.com/apache/datafusion/issues/14634#issuecomment-2654992676
Waiting until we decide on a path forward in https://github.com/apache/datafusion/issues/7765
--
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-unsubscribe@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org
Re: [I] Move `ExpandWildcardRule` into Logical Plan construction [datafusion]
Posted by "jayzhan211 (via GitHub)" <gi...@apache.org>.
jayzhan211 commented on issue #14634:
URL: https://github.com/apache/datafusion/issues/14634#issuecomment-2655172437
I don't think removing Expr::Wildcard is that trivial given there is WildcardOptions and it doesn't block moving `ExpandWildcardRule` out of analyzer
--
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-unsubscribe@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org
Re: [I] Move `ExpandWildcardRule` into Logical Plan construction [datafusion]
Posted by "rkrishn7 (via GitHub)" <gi...@apache.org>.
rkrishn7 commented on issue #14634:
URL: https://github.com/apache/datafusion/issues/14634#issuecomment-2655366067
@jayzhan211 Definitely, I don't think it's blocking either. I guess I was just thinking that if `Expr::Wildcard` gets removed, then it removes the need for this work. But sounds like you're saying it's pretty far away? And that integrating `Analyzer` within the `LogicalPlan` building stage is more immediately doable.
--
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-unsubscribe@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org