jayzhan211 commented on issue #14123: URL: https://github.com/apache/datafusion/issues/14123#issuecomment-2691801390
> I think this sounds like a good idea to me. I don't think we are actively trying to cause pain for downstream users, more like we struggle to find the time to properly think through all potential downstream implications From a developer's perspective, it's easy to overlook how downstream applications use DataFusion. As a result, any changes to it may lead to unexpected impacts. A great way to prevent this is by ensuring that tests or examples cover these use cases within DataFusion. > If Expr::Wildcard no longer works, why is it still being kept around? Expr::Wildcard is still used in other cases not only count(*). We only remove the count wildcard case, so we can't deprecate it until other cases (`select * from table`) are addressed. -- 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...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
